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
Option to preserve T265 maps in memory for subsequent starts #5208
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add it to C# wrapper
Documenting this feature would be useful:
- Default value
- When the option becomes available/disabled
- What happens after n-start/stop operation the map reaches its max. Does it stop update?
include/librealsense2/h/rs_option.h
Outdated
@@ -81,6 +81,7 @@ extern "C" { | |||
RS2_OPTION_ENABLE_POSE_JUMPING, /**< Enable position jumping */ | |||
RS2_OPTION_ENABLE_DYNAMIC_CALIBRATION, /**< Enable dynamic calibration */ | |||
RS2_OPTION_DEPTH_OFFSET, /**< Offset from sensor to depth origin in millimetrers*/ | |||
RS2_OPTION_ENABLE_MAP_PRESERVATION, /**< Preserve previous map when starting */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pls keep the original order and append to end
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for pointing this out. This needs a rebase after 3f82376.
wrappers/matlab/option.m
Outdated
@@ -59,9 +59,11 @@ | |||
enable_relocalization (56) | |||
enable_pose_jumping (57) | |||
enable_dynamic_calibration (58) | |||
enable_depth_offset (59) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check for consistency with c++ declaration
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
…(was the default)
3f82376
to
3d90daa
Compare
…(was the default)
3d90daa
to
c6b56cb
Compare
No description provided.