Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

grid_map_core: Use ament_export_targets and improve eigen linkage #404

Merged
merged 2 commits into from
Feb 5, 2024

Conversation

Ryanf55
Copy link
Collaborator

@Ryanf55 Ryanf55 commented Oct 22, 2023

This fixes include errors in grid_map_geo ros2 port. If you need, I can rebase and target to rolling first.
Relates to #403, but to close that issue out, these changes should ideally be reflected across the whole repo.

@Ryanf55 Ryanf55 changed the title Update ament to latest recommendations grid_map_core: Use ament_export_targets Feb 3, 2024
@Ryanf55 Ryanf55 changed the title grid_map_core: Use ament_export_targets grid_map_core: Use ament_export_targets and improve eigen linkage Feb 3, 2024
@wep21 wep21 self-requested a review February 4, 2024 09:53
Ryanf55 and others added 2 commits February 4, 2024 10:08
* Fixes include errors in grid_map_geo ros2 port

Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
Signed-off-by: wep21 <daisuke.nishimatsu1021@gmail.com>
@wep21
Copy link
Collaborator

wep21 commented Feb 5, 2024

@Ryanf55 This PR includes some commits for rolling. Is it better to change the target branch to rolling?

@Ryanf55 Ryanf55 changed the base branch from humble to rolling February 5, 2024 14:08
@Ryanf55
Copy link
Collaborator Author

Ryanf55 commented Feb 5, 2024

@Ryanf55 This PR includes some commits for rolling. Is it better to change the target branch to rolling?

Ah yes, I've changed the target branch.

Copy link
Collaborator

@wep21 wep21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Ryanf55 Ryanf55 merged commit 274d7c4 into ANYbotics:rolling Feb 5, 2024
@Ryanf55 Ryanf55 deleted the bugfix-403-cmake branch February 5, 2024 14:22
@Ryanf55
Copy link
Collaborator Author

Ryanf55 commented Feb 16, 2024

@Mergifyio backport iron

Copy link

mergify bot commented Feb 16, 2024

backport iron

✅ Backports have been created

Ryanf55 added a commit that referenced this pull request Feb 16, 2024
grid_map_core: Use ament_export_targets and improve eigen linkage (backport #404)
@Ryanf55
Copy link
Collaborator Author

Ryanf55 commented Feb 16, 2024

@Mergifyio backport humble

Copy link

mergify bot commented Feb 16, 2024

backport humble

✅ Backports have been created

Ryanf55 added a commit that referenced this pull request Feb 16, 2024
grid_map_core: Use ament_export_targets and improve eigen linkage (backport #404)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ros2 Affects ROS 2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants