- use IsEqual in stead of == comparison of double

Signed-off-by: Andreas Rauschert <andreas.rb.rauschert@bmw.de>

- added nullptr check for entity
- first try to use the existing velocity vector of the entity to derive
the direction of the future velocity vector before using the entity
orientation

Signed-off-by: Andreas Rauschert <andreas.rb.rauschert@bmw.de>

Signed-off-by: Andreas Rauschert <andreas.rb.rauschert@bmw.de>

Signed-off-by: Andreas Rauschert <andreas.rb.rauschert@bmw.de>

Signed-off-by: Andreas Rauschert <andreas.rb.rauschert@bmw.de>

- using the namespace "units" globally creates a name clash of
units::inverse with the glm library's glm::inverse, which is used within
the glm library. glm lib doesn't compile anymore then
- moved "using namespace units;" to mantle_api namespace for now

Signed-off-by: Andreas Rauschert <andreas.rb.rauschert@bmw.de>

Signed-off-by: Andreas Rauschert <andreas.rb.rauschert@bmw.de>

Signed-off-by: Andreas Rauschert <andreas.rb.rauschert@bmw.de>

- time conversion didn't compile when used
- Added units for spline sections

Signed-off-by: Andreas Rauschert <andreas.rb.rauschert@bmw.de>

Signed-off-by: Andreas Rauschert <andreas.rb.rauschert@bmw.de>

would otherwise cause a compiler error on multiple usages

Change CreateMap parameter to accept a struct, which could be overloaded, i.e. for friction patch support

Signed-off-by: Rene Paris <rene.paris@in-tech.com>

- add 'kStatic' to entity-type enum
- Change IEntityRepository interface for static objects to take a properties pointer

Signed-off-by: Jupp Tscheak <jupp.tscheak@daimler.com>

Signed-off-by: Jupp Tscheak <jupp.tscheak@daimler.com>

Revised the control strategies that are spline based accordingly.

Signed-off-by: Jupp Tscheak <jupp.tscheak@daimler.com>

Type "Time" is now an alias to "units::time::millisecond_t". The conversion functions as provided by "time_utils.h" are still present and revised accordingly."

Signed-off-by: Jupp Tscheak <jupp.tscheak@daimler.com>

would otherwise cause a compiler error on multiple usages

- the "using SimulationTime=..." directive was moved to
Common/time_utils.h and renamed to "using Time=...". Please use this one
- SimulationTime was then redefined in Common/simulation_time.h as
"struct" to hold the simulation time (current&delta)

Signed-off-by: Andreas Rauschert <andreas.rb.rauschert@bmw.de>

- last manual sync before using eclipse repo directly

Signed-off-by: Andreas Rauschert <andreas.rb.rauschert@bmw.de>

Unit is given explicitly by type.

Signed-off-by: Jupp Tscheak <jupp.tscheak@daimler.com>

Replaced the types of all physical quantities related members/arguments with SI unit types as provided by the Units library.

* Underlying type of mantle_api::Vector3 needs to be a unit.
* Underlying type of mantle_api::Dimension3 is SI base unit [m].
* Underlying type of mantle_api::Orientation3 is limited to SI units [rad], [rad/s], [rad/s^2].
* Underlying type of mantle_api::SimulationTime is SI base unit [s].
* Units used in project "Open Simulation Interface (OSI)" were considered accordingly.

Signed-off-by: Jupp Tscheak's avatarJupp Tscheak <jupp.tscheak@daimler.com>

SI unit "second" represented by the Units framework is used for time. Conversion between time based units is handled by the Units framework.

Signed-off-by: Jupp Tscheak's avatarJupp Tscheak <jupp.tscheak@daimler.com>

If scenario domain entities are having references to the same IEntity, it might be that another instance is deleting this IEntity invalidating all references pointing to it. Those situations could be handled using the callback mechanism.

Signed-off-by: Jupp Tscheak <jupp.tscheak@daimler.com>

In order to check whether an IEntity with a certain name/id exists, function IEntityRepository::Contains could be used to query this information. If IEntityRepository::Get is called on a non-existing name/id, the function would throw, so for defensive programming reasons a IEntityRepository::Contains should always be done before actually calling IEntityRepository::Get. The good thing about plain old C-pointers is that assignment and existence check can be done in one call. Since C++17 the same functionality is managed by std::optional.

Another disadvantage of using a plain reference to IEntity is when it is used as a class member, where it needs to be initialized when an object of that class is defined. It might be that during construction time this IEntity is not known/existing yet. Of cource, a plain old C-pointer could be used in this case, but it wouldn't be a consistant usage.

The proposal is to use std::optional<std::reference_wrapper<IEntity>> as return value for IEntityRepository::Get.

Signed-off-by: Jupp Tscheak <jupp.tscheak@daimler.com>