-
Notifications
You must be signed in to change notification settings - Fork 49
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
feat: add autoware_map_msgs for dynamic map loading #39
feat: add autoware_map_msgs for dynamic map loading #39
Conversation
Signed-off-by: kminoda <koji.m.minoda@gmail.com>
Signed-off-by: kminoda <koji.m.minoda@gmail.com>
Signed-off-by: kminoda <koji.m.minoda@gmail.com>
Signed-off-by: kminoda <koji.m.minoda@gmail.com>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
…github.com:kminoda/autoware_msgs into feat/add_autoware_map_msgs_for_dynamic_map_loading
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
a43f673
to
3b76daf
Compare
Signed-off-by: kminoda <koji.minoda@tier4.jp>
Signed-off-by: kminoda <koji.minoda@tier4.jp>
@mitsudome-r @yukkysaito Would you review or approve this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your efforts, it seems good to me.
@kenji-miyake Sorry, would you make this DCO pass again? |
@mitsudome-r @kenji-miyake (mentioning whom seem to have the write access) (Of course any comments instead of the merge is also welcome) |
@kminoda I think @mitsudome-r or @xmfcx is the right person to merge this. |
I've merged it, @mitsudome-r if you have any comments, we can resolve them in another pr. |
Signed-off-by: kminoda koji.m.minoda@gmail.com
Description
Add autoware_map_msgs to integrate a new interface for dynamic map loading. See https://github.com/orgs/autowarefoundation/discussions/2812 for detail.
Related links
The next step of this PR would be to create a dynamic map loading functionality in
map_loader
. Here's how it's going to look like: autowarefoundation/autoware.universe#1938Tests performed
Notes for reviewers
mode
in the response field ofsrv/LoadPCDMaps
?Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.