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

added support for HDL-64E S2.x models #12

Merged
merged 3 commits into from
Jul 26, 2013
Merged

added support for HDL-64E S2.x models #12

merged 3 commits into from
Jul 26, 2013

Conversation

gabor-meszaros
Copy link
Contributor

I added support for HDL-64E S2.x Velodyne models. You can find a test file for HDL-64E S2.1 LIDAR together with the calibration files here: https://dl.dropboxusercontent.com/u/107064156/HDL-64E_S2.1.zip

The connecting issue's link: #11

// Keep the reader from blowing through the file. The
// actual device generates either 2600 (64E) or 1808 (32E)
// packets per second at 600 RPM.
// Keep the reader from blowing through the file.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wanted to avoid the update of this comment every time after you add a new model and the package generation is independent of the packets per second.

The HDL-64E S2 can spin at rates ranging from 300 RPM (5 Hz) to 900 RPM (15 Hz). The
default is 600 RPM (10 Hz). Note that changing the spin rate does not change the data rate
– the unit will send out the same number of packets (at a rate of 1.3 million data points per
second) regardless of spin rate. The image resolution will increase or decrease depending on
rotation speed.

http://www.velodynelidar.com/lidar/products/manual/63-HDL64E%20S2%20Manual_Rev%20D_2011_web.pdf p. 10

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

jack-oquin added a commit that referenced this pull request Jul 26, 2013
added support for HDL-64E S2.x models
@jack-oquin jack-oquin merged commit a351160 into ros-drivers:master Jul 26, 2013
@jack-oquin
Copy link
Member

Looks good. Thanks, Gabor! We would not be able to support these models without your help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants