Utm using geographiclib ros2 branch #833
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ayrton04
reviewed
Oct 25, 2023
|
|
||
| #include "robot_localization/navsat_conversions.hpp" | ||
|
|
||
| void NavsatConversionsTest( |
There was a problem hiding this comment.
Did we confirm that these tests produce the same results before and after the change?
There was a problem hiding this comment.
Yes
But to make it explicit, I opened a separate PR that can be merged first if desired: #838
ayrton04
approved these changes
Oct 25, 2023
HaoguangYang
added a commit
to HaoguangYang/robot_localization
that referenced
this pull request
Mar 26, 2024
* Changelogs * 3.4.0 * Adding issue templates * Update issue templates * install headers (cra-ros-pkg#786) * bump Humble to 3.4.1 for release (cra-ros-pkg#789) * bumping to 3.4.2 for humble release (cra-ros-pkg#806) * bumping humble to 3.5.1 to override incorrect binaries (cra-ros-pkg#816) * Utm using geographiclib humble branch (cra-ros-pkg#834) * Add single test for navsat_conversions * Add a southern point to the navsat_transform test * LLtoUTM using GeographicLib * Use GeographicLib for UTMtoLL conversions * Linting * Forgot include * Fix compilation * Calculate gamma because it's a function output and was supplied before * Also test for gamma conversion * Align naming and install * Utm using geographiclib ros2 branch (cra-ros-pkg#833) * Add single test for navsat_conversions * Add a southern point to the navsat_transform test * LLtoUTM using GeographicLib * Use GeographicLib for UTMtoLL conversions * Linting * Forgot include * Fix compilation * Calculate gamma because it's a function output and was supplied before * Also test for gamma conversion * Align naming and install * Test navsat transform functionality (cra-ros-pkg#838) * Wait for odometry message before setting manual datum so that the base and world frame names can be set. (cra-ros-pkg#835) * wait for odom msg before setting manual datum * Wait for odometry message before setting manual datum so that the base and world frame names can be set. (cra-ros-pkg#836) * wait for odom msg before setting manual datum * fix header timestamp (cra-ros-pkg#852) Co-authored-by: Luke Chang <luke@boxfish.nz> * fix header timestamp (cra-ros-pkg#852) Co-authored-by: Luke Chang <luke@boxfish.nz> * Changelogs * 3.5.2 * Fixing angle clamping for humble (cra-ros-pkg#854) * fix: modify dual_ekf_navsat_example.launch file to remap the correct imu topic (cra-ros-pkg#857) * Feature/set utm service (cra-ros-pkg#856) * Forward port Fix/set utm map frame change --------- Signed-off-by: Tim Clephas <tim.clephas@nobleo.nl> * simplified calculation * simplified calculation * Update issue templates * Migrate static tfs to ros2 format. (cra-ros-pkg#864) * Fix throttle duration (cra-ros-pkg#866) * Fix throttle duration * Update ukf.yaml to match ekf.yaml (cra-ros-pkg#867) Add missing *_pose_use_child_frame parameter. * updated file formatting --------- Signed-off-by: Tim Clephas <tim.clephas@nobleo.nl> Co-authored-by: Tom Moore <tmoore@locusrobotics.com> Co-authored-by: Olivier Kermorgant <olivier_kermorgant@yahoo.fr> Co-authored-by: Steve Macenski <stevenmacenski@gmail.com> Co-authored-by: Tim Clephas <tim.clephas@nobleo.nl> Co-authored-by: Tom Greier <tgreier@moog.com> Co-authored-by: Luke Chang <luke.x.chang@gmail.com> Co-authored-by: Luke Chang <luke@boxfish.nz> Co-authored-by: joeldushouyu <65317431+joeldushouyu@users.noreply.github.com> Co-authored-by: Mukunda Bharatheesha <mukunda.bharatheesha@nobleo.nl> Co-authored-by: thandal <than@timbrel.org>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Extracting the answer from this question: #830 (comment) from this comment: #821 (comment)