Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

minor change in LidarVisual for the case when sensor data is more than sensor range #132

Merged
merged 1 commit into from
Aug 26, 2020

Conversation

mihirk284
Copy link
Contributor

This PR attempts to fix the case when the data given to the sensor happens to contain a range value which is greater than, or exactly equal to the sensor range.
For rays when the sensor does not hit anything in the range, it returns inf for the ray. However, when we add adding noise to the measurements defaults the inf reading exactly equal to maxRange of the sensor. This causes the lidar visual to display the rays, although they are not hitting any object.

Additionally, if a point is a finite value (even if it is greater than the maxRange), the LidarVisual showed it since the only criteria of determining if the sensor ray was hitting or not was if the value returned by the sensor was inf.

Signed-off-by: Mihir Kulkarni <mihirk284@gmail.com>
@mihirk284 mihirk284 requested a review from iche033 as a code owner August 26, 2020 09:36
Copy link
Contributor

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

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

LGTM

@chapulina chapulina added the 🔮 dome Ignition Dome label Aug 26, 2020
@iche033 iche033 merged commit d765aba into gazebosim:master Aug 26, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🔮 dome Ignition Dome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants