search

ethz-asl / wavemap

Fast, efficient and accurate multi-resolution, multi-sensor 3D occupancy mapping
https://ethz-asl.github.io/wavemap/
BSD 3-Clause "New" or "Revised" License
447 stars 38 forks source link

Missing frame id and stamp in msgs #26

Closed astumpf closed 1 year ago

astumpf commented 1 year ago

The latest changes introduced a minor error. When using non-hashed octree the map msg is missing header information. The code here should be

      if (convert::mapToRosMsg(*occupancy_map_, config_.world_frame,
                              ros::Time::now(), map_msg)) {
victorreijgwart commented 1 year ago

Hi Alexander, Thanks for spotting this! The suggested fix looks good. Shall I open a PR for it, or would you like to do it?

astumpf commented 1 year ago

Opened a pr: https://github.com/ethz-asl/wavemap/pull/27

victorreijgwart commented 1 year ago

Thanks!