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.

Sign up for GitHub

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

Jump to bottom

velodyne_pointcloud: dynamic reconfigure limits min_range to 0.9 meters #60

Closed

jack-oquin opened this issue Oct 14, 2015 · 5 comments

Member

jack-oquin commented Oct 14, 2015

Some models may support shorter ranges, so configuration should allow smaller settings.

Maybe this is only a theoretical problem, but it prevents people from experimenting.

Member Author

jack-oquin commented Oct 14, 2015

Apparently, the VLP-16 is limited to 0.9 meters, like most of the other models.

At some point there was an attempt to set min_range to 0.4 in the VLP-16 launch script, but due to bugs in that script and in dynamic reconfiguration it was not having any effect. With those bugs fixed, there are still no points returned from the device at less than about 0.9 meters range.

jack-oquin added a commit that referenced this issue


          permit min_range settings below 0.9 meters (#60)

d65ef88

No known models are currently known to return closer measurements.

jack-oquin closed this as completed

jack-oquin referenced this issue


          adding support for the VLP16

0235e9f

Member Author

jack-oquin commented Oct 14, 2015

The Windows VeloView program does show VLP-16 returns as short as 0.45 meters.

So, there must be some additional bug in this driver.

jack-oquin reopened this

jack-oquin added a commit that referenced this issue


          restore VLP-16 min_range setting to 0.4 (#60)

85228ed

NOTE: There is still some other problem keeping that from working.

Member Author

jack-oquin commented Oct 14, 2015

The VLP-16 manual on page 3 does mention a "range from 1m to 100m".

piyushk commented Oct 14, 2015

@jack-oquin Thanks for finding that. I missed it in my original sweep.

I'll perform additional testing tomorrow to see if I can find a solution to this problem.

jack-oquin mentioned this issue

VLP-16: VeloView returns data at shorter range than this driver #61

Closed

Member Author

jack-oquin commented Oct 14, 2015

This issue is fixed. The remaining problem with close-range VLP-16 data deserves a separate issue, #61.

jack-oquin closed this as completed

vrabaud pushed a commit to vrabaud/velodyne that referenced this issue


          permit min_range settings below 0.9 meters (ros-drivers#60)

b60e7e9

No known models are currently known to return closer measurements.

vrabaud pushed a commit to vrabaud/velodyne that referenced this issue


          restore VLP-16 min_range setting to 0.4 (ros-drivers#60)

07b708c

NOTE: There is still some other problem keeping that from working.

vrabaud pushed a commit to vrabaud/velodyne that referenced this issue


          permit min_range settings below 0.9 meters (ros-drivers#60)

490376c

No known models are currently known to return closer measurements.

vrabaud pushed a commit to vrabaud/velodyne that referenced this issue


          restore VLP-16 min_range setting to 0.4 (ros-drivers#60)

78094e4

NOTE: There is still some other problem keeping that from working.

vrabaud pushed a commit to vrabaud/velodyne that referenced this issue


          permit min_range settings below 0.9 meters (ros-drivers#60)

cc943e3

No known models are currently known to return closer measurements.

vrabaud pushed a commit to vrabaud/velodyne that referenced this issue


          restore VLP-16 min_range setting to 0.4 (ros-drivers#60)

eb25a92

NOTE: There is still some other problem keeping that from working.

vrabaud pushed a commit to vrabaud/velodyne that referenced this issue


          permit min_range settings below 0.9 meters (ros-drivers#60)

65a2180

No known models are currently known to return closer measurements.

vrabaud pushed a commit to vrabaud/velodyne that referenced this issue


          restore VLP-16 min_range setting to 0.4 (ros-drivers#60)

57463bc

NOTE: There is still some other problem keeping that from working.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment