feat(map_loader): add support for local cartesian projection to lanele…

[sebekx]

lanelet map loader

Description

Add LocalCaretsian (WGS84) projection support to map_loader.

Requires:

• feat(tier4_map_msgs): add LocalCartesian const tier4/tier4_autoware_msgs#152

Related links

autoware.core PR
autoware_msgs PR
Parent Issue:
None.

How was this PR tested?

Used map_loaded to load lanelet with localCartesian(WGS84) projection.

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

[github-actions]

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

• You've checked our contribution guidelines.
• Your PR follows our pull request guidelines.
• All required CI checks pass before marking the PR ready for review.

[TaikiYamada4]

@sebekx
Could you please share your thoughts why you recommend this change?
Adding a new mapping projection doesn't only influence the map_loader but also make effects to other modules such as gnss_poser, MapProjectorInfo.msg, and maybe more.

Since we cannot discuss or estimate the impact to the entire Autoware enough here, I would recommend you to create a discussion about adding LocalCartesian support to Autoware.

[sebekx]

@sebekx Could you please share your thoughts why you recommend this change? Adding a new mapping projection doesn't only influence the map_loader but also make effects to other modules such as gnss_poser, MapProjectorInfo.msg, and maybe more.

Since we cannot discuss or estimate the impact to the entire Autoware enough here, I would recommend you to create a discussion about adding LocalCartesian support to Autoware.

We are using LocalCartesian in our stack with different implementation of localization components. Adding it in here shouldn't affect the system unless someone use it.

[TaikiYamada4]

@sebekx

We are using LocalCartesian in our stack with different implementation of localization components. Adding it in here shouldn't affect the system unless someone use it.

If so, then please add the information in README.md that LocalCartesian can only be used in lanelet2_map_loader and packages like gnss_poser doesn't support it.

Then I suppose the changes here is OK, but it's only in my sense.
I'm not sure what the reviewers of the MapProjectorInfo.msg PR might say.

[sebekx]

If so, then please add the information in README.md that LocalCartesian can only be used in lanelet2_map_loader and packages like gnss_poser doesn't support it.

Ok, thanks, I have updated readme.

[TaikiYamada4]

@sebekx

Since map_loader will move to Autoware.Core, you shall resolve the conflicts here and the conversations happening in the PR of tier4_map_msgs soon if this is urgent.
Unless, there might be some other tasks coming up after map_loader is moved to Autoware.Core

[xmfcx]

• @sebekx can you rebase this PR after updating feat(tier4_map_msgs): add LocalCartesian const tier4/tier4_autoware_msgs#152 ?

[sebekx]

• @sebekx can you rebase this PR after updating feat(tier4_map_msgs): add LocalCartesian const tier4/tier4_autoware_msgs#152 ?

rebased and created pr:

• autoware.core PR
• autoware_msgs PR