Skip to content
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

OS time synchronization with external GNSS #209

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

retepik
Copy link

@retepik retepik commented Jan 21, 2021

Hello,
I added few parameters to configure client. This parameters are useful for time synchronization OS sensor with GNSS (GPS) receiver or INS (I tested it with Advanced navigation Spatial Fog Dual), for scanning window setting and for record of packets in ROS. I added this parameters also into ouster.launch file.

Peter Jurasko added 2 commits January 19, 2021 17:28
…IDAR with GNSS receiver, add parameters to config scanning window, add parameters to record data in ros, modify ouster.launch to reflect changes in client
@dmitrig
Copy link
Contributor

dmitrig commented Jan 4, 2022

Thanks for the contribution! Having some way of setting the rest of the configuration through ROS has been on our backlog for a while. Would it be good enough to allow specifying a config file as a parameter to ouster.launch vs. threading each parameter through?

@dmitrig dmitrig added ROS Sample ROS driver enhancement New feature or request labels Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ROS Sample ROS driver
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants