Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Solving the problem of the same vector manipulation function being implemented in multiple locations.
Abstract
Fixes:
lenear_albebra
. Consequently, revised/geometry/vector3/operator.hpp
to support lenear_albebra-specific features.lenear_albebra
was interfering, so it was migrated to operator.hpp.getInternalAngle
andinnerProduct
functions inlenear_albebra
could not be used, so they were moved to/geometry/vector3/
and templated.lenear_albebra
was removed because it had the same functionality asnorm
.ScenarioSimulator
.Background
Issue: #1256
Details
These newly added files are templated versions of functions that were in
linear_algebra
. The functionality itself has not been changed.common/math/geometry/include/geometry/vector3/inner_product.hpp
common/math/geometry/include/geometry/vector3/internal_angle.hpp
common/math/geometry/include/geometry/vector3/vector3.hpp
In
common/math/geometry/include/geometry/vector3/is_like_vector3.hpp
, the following additions were made:w
is not included as an element.IsLikeVector3
to exclude Quaternions.IsLikeQuaternion
to detect Quaternions.In
common/math/geometry/include/geometry/vector3/operator.hpp
, the following changes were made:Point
type when overloading operators.Tests were modified to accommodate these changes. There is no change in the coverage rate.
To accommodate these changes, the following files were modified:
common/math/geometry/src/polygon/line_segment.cpp
common/math/geometry/src/spline/catmull_rom_spline.cpp
simulation/do_nothing_plugin/src/plugin.cpp
simulation/traffic_simulator/src/behavior/follow_trajectory.cpp
simulation/traffic_simulator/src/behavior/longitudinal_speed_planning.cpp
simulation/traffic_simulator/src/entity/entity_manager.cpp
simulation/traffic_simulator/src/hdmap_utils/hdmap_utils.cpp
References
https://github.com/OUXT-Polaris/quaternion_operation/blob/2c0bf2a60c50262921f9f1858e474030a39560a0/src/quaternion_operation.cpp#L30-L50
Destructive Changes
N/A
Known Limitations
N/A