autoware.universe icon indicating copy to clipboard operation
autoware.universe copied to clipboard

Proposal for Integration and Contribution of Ouster Lidar and DURO GNSS to Autoware Repository

Open zeta-mobility opened this issue 10 months ago • 3 comments

Checklist

  • [X] I've read the contribution guidelines.
  • [X] I've searched other issues and no duplicate issues were found.
  • [X] I've agreed with the maintainers that I can plan this task.

Description

This issue concerns the integration and possible contribution of external sensors for Ouster Lidar and DURO GNSS, which are not currently available in the Autoware repository.

Purpose

  • Ouster Lidar: Documented as 'Autoware tested' on Autoware's LIDAR documentation page, yet no corresponding sensor integration code is available in the GitHub repository. We have developed our own integration code for the Ouster Lidar sensor and would like to discuss the possibility of contributing it to the repository.
  • DURO GNSS: We have successfully integrated DURO GNSS with Autoware and are prepared to contribute our integration code to the software, despite it not being officially supported.

Possible approaches

  1. Review and discussion with maintainers about the interest and requirements for incorporating these ideas and code into the main Autoware repository.
  2. If agreed, proceed with pull requests for each component.
  3. Update documentation to reflect the addition of these new integrations.
  4. Possible adjustments or extensions of CI to include these new components.

Definition of done

  • Clarification from maintainers regarding the contribution of Ouster Lidar and DURO GNSS integration.
  • Acceptance of pull requests for both integration ideas, if approved.
  • Documentation and CI updates to include these new integration approaches.

zeta-mobility avatar Apr 18 '24 16:04 zeta-mobility

Ouster Lidar: Documented as 'Autoware tested' on Autoware's LIDAR documentation page, yet no corresponding sensor integration code is available in the GitHub repository. We have developed our own integration code for the Ouster Lidar sensor and would like to discuss the possibility of contributing it to the repository.

Is you implementation anything different from https://autowarefoundation.github.io/autoware-documentation/main/reference-hw/lidars/? Also, there is a work in Sensing and Perception Working Group to develop unified lidar driver for different vendors. Do you think your implementation fits to the design? (cc. @drwnz )

mitsudome-r avatar Apr 23 '24 09:04 mitsudome-r

Thank you for the response @mitsudome-r .

To clarify, we have not developed the driver itself; instead, we implement the code that converts the data received from the driver into the format required by Autoware's sensing pipeline. This transformation enables seamless integration of the data as an input to the sensing system.

Thus, both the LiDAR(Ouster) and GNSS(Swift Duro) data were integrated not through the development of new drivers, but by modifying the data format to ensure compatibility with the existing system.

zeta-mobility avatar Apr 25 '24 11:04 zeta-mobility

Hello @zeta-mobility . To utilize GNSS, you'll need to adjust your message type to sensor_msgs/NavSatFix. I believe there's no need for additional additions as Autoware is presently compatible with this ROS default message type.

meliketanrikulu avatar May 09 '24 15:05 meliketanrikulu

This pull request has been automatically marked as stale because it has not had recent activity.

stale[bot] avatar Jul 08 '24 22:07 stale[bot]

Hi @zeta-mobility Are there any updates?

We have developed our own integration code for the Ouster Lidar sensor and would like to discuss the possibility of contributing it to the repository.

Recently Autoware updated the point types and we have observed that some users are struggling with vendor drivers. If you're still interested in contributing, we'd like to encourage you to integrate Ouster sensors into Nebula driver 🚀

amadeuszsz avatar Jul 31 '24 02:07 amadeuszsz

This pull request has been automatically marked as stale because it has not had recent activity.

stale[bot] avatar Sep 29 '24 12:09 stale[bot]