-
Notifications
You must be signed in to change notification settings - Fork 65
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(map_projection_loader): add scale_factor and remove altitude #340
feat(map_projection_loader): add scale_factor and remove altitude #340
Conversation
Signed-off-by: Yamato Ando <yamato.ando@tier4.jp>
Signed-off-by: Yamato Ando <yamato.ando@tier4.jp>
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
map/autoware_map_projection_loader/src/map_projection_loader.cpp
Outdated
Show resolved
Hide resolved
Co-authored-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
Signed-off-by: Yamato Ando <yamato.ando@tier4.jp>
Signed-off-by: Yamato Ando <yamato.ando@tier4.jp>
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.
LGTM
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.
I have confirmed that the logging_simulator work well.
LGTM
The dependencies are ready. |
Signed-off-by: Yamato Ando <yamato.ando@tier4.jp>
There was an error in the test of autoware_geograhpy_utils. Fixed in b20c158 |
I've manually removed the GitHub Actions' cache and hope it would be resolved. |
Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
I investigated various aspects but couldn’t find the issue. I added a one-line comment to ignore the clang-tidy error and a TODO comment. 1fd7a35 |
This reverts commit e9af822.
Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
According to https://github.com/autowarefoundation/autoware_msgs/blob/main/autoware_map_msgs/msg/MapProjectorInfo.msg#L26, the |
Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
I gave up for today. I don’t understand why
Kindly ping @veqcc @mitsudome-r |
I've found the issue. Here the clang-tidy job is installing dependency Why is it doing that?Here it tries to pull the build-depend.repos but we removed them, so it cannot find them. And underlay-workspace is not sourced. How to fix?Add underlay-workspace arg to the clang-tidy/action.yaml and source it before performing the |
@xmfcx Great job!!!! |
Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
After merging autowarefoundation/autoware-github-actions#341 and reverting d520a70, we will finally merge this. |
@youtalk Possible general (not specific to this case) solutions are
|
Merged finally. Thank you for your all support! |
Thank you all!! |
Description
add scale_factor and remove altitude.
Related links
https://github.com/orgs/autowarefoundation/discussions/5765
Parent Issue:
How was this PR tested?
Check scale factor
Confirmed that it works correctly as usual.
I have prepared a sample map converted to the Transverse Mercator coordinate system with different scale factor.
I have prepared the PCD map and lanelet2 map in each folder so that their scale factors match. If you open
map_projector_info.yaml
, change thescale_factor
, and then run Autoware, only the lanelet2 map will be converted according to the modified scale factor.However, position estimation will fail with any value other than the original scale factor (0.9996).
Please verify that the map changes according to the scale factor.
examples
pcd map: original scale factor (0.9996)

lanelet2 map: original scale factor (0.9996)
pcd map: original scale factor (0.9996)

lanelet2 map: scale factor 1.5
Check altitude
Since this change has no impact, PR test is not required.
Notes for reviewers
Please merge this PR after autowarefoundation/autoware_msgs#121 and autowarefoundation/autoware_lanelet2_extension#54.
Interface changes
None.
Effects on system behavior
None.