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.

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

Add Max Usable Range feature #7637

Merged
merged 14 commits into from
Oct 26, 2020
Merged

Add Max Usable Range feature #7637

merged 14 commits into from
Oct 26, 2020

Conversation

Nir-Az
Copy link
Collaborator

@Nir-Az Nir-Az commented Oct 22, 2020 •
edited
Loading

Tracked on [RS5-9263]

Max Usable Range calculates the maximum range of the camera given the amount of ambient light in the scene.

  • Added to Viewer + LibRS API (not DQT)
  • New option : RS2_OPTION_MAX_USABLE_RANGE
  • New LibRS API function "rs2_get_max_usable_range"
  • Add sensor extension "RS2_EXTENSION_MAX_USABLE_RANGE_SENSOR"
  • Wrappers handling (tested on C#, Python, C++)

Support FW version >= 1.5.0.0 (added NEST (Noise-Estimation) value to temperatures)

image

maloel
maloel reviewed Oct 22, 2020
View reviewed changes
src/l500/l500-depth.cpp Outdated Show resolved Hide resolved
maloel
maloel reviewed Oct 22, 2020
View reviewed changes
src/l500/l500-device.cpp Outdated Show resolved Hide resolved
maloel
maloel reviewed Oct 22, 2020
View reviewed changes
common/model-views.cpp Outdated Show resolved Hide resolved
maloel
maloel reviewed Oct 26, 2020
View reviewed changes
common/model-views.cpp Outdated Show resolved Hide resolved
@maloel maloel merged commit f1f5eeb into IntelRealSense:development Oct 26, 2020
@Nir-Az Nir-Az deleted the add_max_range_tool branch December 23, 2020 07:24
@RealSenseSupport RealSenseSupport mentioned this pull request Jan 11, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maloel maloel maloel approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Nir-Az @maloel