Skip to content

Set depth invalidation enabled by CMake flag - #6288

Merged
dorodnic merged 4 commits into
realsenseai:developmentfrom
Nir-Az:change_depth_invalidation_enabled_v2
Apr 23, 2020
Merged

Set depth invalidation enabled by CMake flag#6288
dorodnic merged 4 commits into
realsenseai:developmentfrom
Nir-Az:change_depth_invalidation_enabled_v2

Conversation

@Nir-Az

@Nir-Az Nir-Az commented Apr 22, 2020

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread src/l500/CMakeLists.txt Outdated
)

if (ENABLE_DEPTH_INVALIDATION)
add_definitions(-DDEPTH_INVALIDATION_ENABLED)

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.

I suggest you keep the define name the same as the CMake variable to make it more intuitive to search

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

agreed.

Comment thread src/l500/l500-depth.h Outdated
});

register_option(static_cast<rs2_option>(RS2_OPTION_DEPTH_INVALIDATION_ENABLE), _depth_invalidation_option);
// The depth invalidation enable option is depricated for now.

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.

deprecated

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

agreed.

Comment thread CMake/lrs_options.cmake Outdated
option(FORCE_WINUSB_UVC "Explicitly turn-on winusb_uvc (for win7) backend - deprecated, use FORCE_RSUSB_BACKEND instead" OFF)
option(ANDROID_USB_HOST_UVC "Build UVC backend for Android - deprecated, use FORCE_RSUSB_BACKEND instead" OFF) No newline at end of file
option(ANDROID_USB_HOST_UVC "Build UVC backend for Android - deprecated, use FORCE_RSUSB_BACKEND instead" OFF)
option(ENABLE_DEPTH_INVALIDATION "Turn on the depth frame validator" ON)

@maloel maloel Apr 23, 2020

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.

Might want to mention it's for L500 only... maybe make it ENABLE_L500_DEPTH_INVALIDATION?
Also, it's not exactly clear what "depth invalidation" is -- maybe mention "to automatically catch corrupted frames and restart the sensor"?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

agreed.

@dorodnic
dorodnic merged commit 7a39551 into realsenseai:development Apr 23, 2020
@Nir-Az
Nir-Az deleted the change_depth_invalidation_enabled_v2 branch April 23, 2020 12:19
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