Bug fix: RobotTrajectory append() (backport #1813)

[mergify]

This is an automatic backport of pull request #1813 done by Mergify.

---

Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

• @Mergifyio refresh will re-evaluate the rules
• @Mergifyio rebase will rebase this PR on its base branch
• @Mergifyio update will merge the base branch into this PR
• @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

• look at your merge queues
• generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

[codecov]

Codecov Report

Patch coverage: 64.05% and project coverage change: +0.02% 🎉

Comparison is base (05f3c6d) 50.93% compared to head (e3982b7) 50.95%.
Report is 15 commits behind head on humble.

Additional details and impacted files

@@            Coverage Diff             @@
##           humble    #1821      +/-   ##
==========================================
+ Coverage   50.93%   50.95%   +0.02%     
==========================================
  Files         381      382       +1     
  Lines       31884    31895      +11     
==========================================
+ Hits        16238    16248      +10     
- Misses      15646    15647       +1     

Files Changed	Coverage Δ
moveit_core/kinematic_constraints/src/utils.cpp	24.53% <0.00%> (ø)
...oveit/online_signal_smoothing/butterworth_filter.h	100.00% <ø> (ø)
...ene/include/moveit/planning_scene/planning_scene.h	43.55% <ø> (ø)
...pl/ompl_interface/src/planning_context_manager.cpp	53.31% <0.00%> (-1.05%)	⬇️
.../moveit_servo/include/moveit_servo/pose_tracking.h	100.00% <ø> (ø)
...eit/planning_scene_monitor/current_state_monitor.h	100.00% <ø> (ø)
...g/rdf_loader/src/synchronized_string_parameter.cpp	32.31% <0.00%> (-0.50%)	⬇️
...it_ros/robot_interaction/src/robot_interaction.cpp	0.00% <0.00%> (ø)
...obot_state_rviz_plugin/src/robot_state_display.cpp	0.00% <0.00%> (ø)
...ugin_render_tools/src/trajectory_visualization.cpp	0.00% <0.00%> (ø)
... and 7 more

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[henningkayser]

@AndyZe any idea what's wrong with the tests?

[github-actions]

This PR is stale because it has been open for 45 days with no activity. Please tag a maintainer for help on completing this PR, or close it if you think it has become obsolete.

[sea-bass]

I just found that this PR not being backported because of the stale bot closing it caused a fix to the Pilz sequencing (#2961) (which was backported) to regress on Humble.

See #3161 (comment)

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 51.25%. Comparing base (06184b2) to head (2e24608).
Report is 1 commits behind head on humble.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##           humble    #1821      +/-   ##
==========================================
- Coverage   51.25%   51.25%   -0.00%     
==========================================
  Files         382      382              
  Lines       31885    31885              
==========================================
- Hits        16341    16340       -1     
- Misses      15544    15545       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.