autowarefoundation / autoware

Autoware - the world's leading open-source software project for autonomous driving
https://www.autoware.org/
Apache License 2.0
9.15k stars 3.06k forks source link

Add option to lanelet2_map_loader for local coordinates #3427

Closed isamu-takagi closed 11 months ago

isamu-takagi commented 1 year ago

Checklist

Description

I'm developing API that provides the global vehicle position (latitude and longitude). For that, I tried to convert from map position to global position, but the autoware osm parser in the lanelet2_extension package overwrites position if local_x or local_y is specified in the Lanelet2 map. This works regardless of the projection type settings in the map loader, the transformation may not match the map projection type. So add an explicit option to the map loader to use local coordinates.

Purpose

To support reverse projection of coordinates in different nodes

Possible approaches

Definition of done

tkhmy commented 1 year ago

I removed the local parser for lanelet_extension https://github.com/autowarefoundation/autoware_common/pull/174

the local parser will be in map_loader https://github.com/autowarefoundation/autoware.universe/pull/3492

xmfcx commented 1 year ago

@isamu-takagi -san,

The task: Add dummy projector to lanelet2_extension

isamu-takagi commented 1 year ago

The task: Add dummy projector to lanelet2_extension

This task is not important. Currently, if a local type is used, it is overwritten with local coordinates after being projected with MGRS, so there is a bit of processing waste.

Also, there may be cases where only some positions are overwritten with local coordinates. In this case, it is better to be able to specify a sub-projection type when using local projections.

isamu-takagi commented 11 months ago

Also, there may be cases where only some positions are overwritten with local coordinates. In this case, it is better to be able to specify a sub-projection type when using local projections.

Changed to use dummy projector. I will consider adding the above cases if necessary. https://github.com/autowarefoundation/autoware.universe/pull/5866

xmfcx commented 11 months ago

Thank you @isamu-takagi -san!