Skip to content

Added visual presets for L500 - #5741

Merged
ev-mp merged 10 commits into
realsenseai:developmentfrom
aangerma:l500_controls
Feb 6, 2020
Merged

ev-mp merged 10 commits into
realsenseai:developmentfrom
aangerma:l500_controls

Conversation

@aangerma

@aangerma aangerma commented Jan 29, 2020 •

Copy link
Copy Markdown
Contributor

Added support for new options in L515:
confidence,
post_processing_sharpness ,
pre_processing_sharpness ,
noise_filtering ,
avalanche_photo_diode
laser_gain,
min_distance ,
invalidation_bypass,
ambient_light

Added visual presets for L500:
RS2_L500_VISUAL_PRESET_NO_AMBIENT,
RS2_L500_VISUAL_PRESET_LOW_AMBIENT,
RS2_L500_VISUAL_PRESET_MAX_RANGE,
RS2_L500_VISUAL_PRESET_SHORT_RANGE

and added support on save and load json for these options.

Tracked on: RS5-6455

@aangerma
aangerma requested review from dorodnic, ev-mp and maloel January 29, 2020 15:08
@aangerma
aangerma force-pushed the l500_controls branch 2 times, most recently from 50bc552 to bc277fe Compare January 30, 2020 09:48

@ev-mp ev-mp left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Add JIRA reference to PR Description
Python/C# for .def changes

Comment thread common/model-views.cpp Resolved
Comment thread src/l500/l500-depth.cpp Resolved
Comment thread src/l500/l500-private.cpp Resolved
Comment thread src/l500/l500-private.h Resolved
Comment thread src/option.h Resolved
Comment thread src/types.cpp Resolved
Comment thread unit-tests/unit-tests-live.cpp Resolved
Comment thread src/l500/l500-device.h Resolved
Comment thread src/realsense.def
rs2_set_amp_factor
rs2_rs400_visual_preset_to_string
rs2_l500_visual_preset_to_string
rs2_sensor_mode_to_string

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Needs Python/C# version

Comment thread common/model-views.h Resolved
Comment thread common/model-views.h Resolved
Comment thread include/librealsense2/h/rs_option.h Resolved
Comment thread include/librealsense2/h/rs_option.h Resolved
Comment thread src/serializable.h Resolved
Comment thread src/serializable.h Resolved
Comment thread src/l500/l500-serializable.h Resolved
Comment thread src/option.h Resolved
Comment thread src/option.h Resolved
Comment thread src/option.h Resolved
Comment thread src/option.h Resolved
Comment thread common/model-views.cpp Resolved
Comment thread common/model-views.cpp Resolved
Comment thread include/librealsense2/h/rs_option.h Resolved
Comment thread include/librealsense2/h/rs_option.h Resolved
Comment thread include/librealsense2/h/rs_option.h Resolved
Comment thread src/serializable-interface.h Resolved
Comment thread include/librealsense2/h/rs_option.h Resolved
Comment thread src/types.cpp Resolved
Comment thread unit-tests/unit-tests-live.cpp Resolved
Comment thread unit-tests/unit-tests-live.cpp Resolved
Comment thread unit-tests/unit-tests-live.cpp Resolved
Comment thread unit-tests/unit-tests-live.cpp Resolved
Comment thread unit-tests/unit-tests-live.cpp Resolved
Comment thread src/l500/l500-depth.cpp Resolved
Comment thread src/l500/l500-serializable.cpp Resolved
Comment thread src/rs.cpp Resolved
Comment thread src/rs.cpp Resolved
Comment thread src/l500/l500-device.h Resolved
Comment thread src/l500/l500-options.cpp Resolved
Comment thread common/model-views.cpp Resolved
Comment thread src/l500/l500-depth.cpp Resolved
Comment thread src/l500/l500-factory.cpp Resolved
# Conflicts:
#	common/on-chip-calib.cpp
#	include/librealsense2/h/rs_option.h
#	src/l500/l500-private.h
#	src/types.cpp
Comment thread src/l500/l500-options.cpp Outdated
if (std::find(advanced_controls.begin(), advanced_controls.end(), opt) != advanced_controls.end())
move_to_custom ();
else
throw wrong_api_call_sequence_exception(to_string() << "on_set_option support advanced controlls only "<< opt<<" injected");

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

controls

@ev-mp
ev-mp merged commit 4e4af79 into realsenseai:development Feb 6, 2020
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.

3 participants