You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I created Trajectory class in autoware_motion_utils.
This class provides a new way of manipulating trajectory, and will alternate current motion_utils's functions.
Currently, these classes is implemented in autoware_motion_utils. However I think this class should be implemented in different package from autoware_motion_utils.
The reasons are listed below.
Trajectory has no dependencies with other programs in autoware_motion_utils.
Trajectory's code is too large.
Namespace can be shortened by removing ::motion_utils::.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
I created
Trajectory
class inautoware_motion_utils
.This class provides a new way of manipulating trajectory, and will alternate current motion_utils's functions.
Currently, these classes is implemented in autoware_motion_utils. However I think this class should be implemented in different package from
autoware_motion_utils
.The reasons are listed below.
::motion_utils::
.I created PR about this.
autowarefoundation/autoware.universe#9253
If you have an opinion on this, please let us know.
Beta Was this translation helpful? Give feedback.
All reactions