IntelRealSense / librealsense

Intel® RealSense™ SDK
https://www.intelrealsense.com/
Apache License 2.0
7.63k stars 4.83k forks source link

Table read error fallback #13512

Closed Nir-Az closed 1 week ago

Nir-Az commented 1 week ago

We wish to allow the camera to enumerate even no flash tables. image

Tracked on [RSDEV-2874] [RSDEV-2901]

Nir-Az commented 1 week ago

Notes from @ev-mp

  1. Can you add a note whether this applies to RGB or Depth sensor intrinsic for clarity?
  2. What is the impact of returning empty intrinsic ? Providing arbitrary values may mask the issue making the user unaware of the actual problem
Nir-Az commented 1 week ago

Notes from @ev-mp

  1. Can you add a note whether this applies to RGB or Depth sensor intrinsic for clarity?
  2. What is the impact of returning empty intrinsic ? Providing arbitrary values may mask the issue making the user unaware of the actual problem
  1. Yes will do
  2. The user will get a log error, if you prefer I can have an internal flag and override the start stream function and throw on this case, not sure it is worth it. thoughts?