Skip to content

fix vec bug, added support for PoseWithCovariance - #24

Merged
jorgenfj merged 1 commit into
mainfrom
fix/ros-conversion-vec-return-type
Dec 21, 2025
Merged

jorgenfj merged 1 commit into
mainfrom
fix/ros-conversion-vec-return-type

Conversation

@jorgenfj

Copy link
Copy Markdown
Contributor

Fixed a bug where ros_to_pose_vec would not return a vector.
Added suport for PoseWithCovariance
Added explicit tests for all supported types

@jorgenfj
jorgenfj requested a review from Andeshog December 21, 2025 18:28
@codecov

codecov Bot commented Dec 21, 2025 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.06%. Comparing base (9bb93fa) to head (3a28c61).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
+ Coverage   97.04%   97.06%   +0.02%     
==========================================
  Files           5        5              
  Lines         440      443       +3     
  Branches      103      106       +3     
==========================================
+ Hits          427      430       +3     
  Misses          6        6              
  Partials        7        7              
Flag Coverage Δ
unittests 97.06% <100.00%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
include/vortex/utils/ros_conversions.hpp 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jorgenfj
jorgenfj merged commit c89beaf into main Dec 21, 2025
5 checks passed
@jorgenfj
jorgenfj deleted the fix/ros-conversion-vec-return-type branch December 21, 2025 22:38
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.7.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants