Skip to content

L515 - Add temperature fetcher thread - #7600

Merged
maloel merged 3 commits into
realsenseai:developmentfrom
Nir-Az:add_get_temp_thread
Oct 21, 2020
Merged

maloel merged 3 commits into
realsenseai:developmentfrom
Nir-Az:add_get_temp_thread

Conversation

@Nir-Az

@Nir-Az Nir-Az commented Oct 18, 2020

Copy link
Copy Markdown
Collaborator

L515 only

Periodic read of temperatures and noise estimation values once depth sensor is on.
a precondition for reflectivity tool + max range [RS5-9263]

Description:

  • A new temperature fetcher thread is created on depth sensor start and closed on stop.
  • All of temperature clients get it from a "get_temperatures" new function
  • get_temperatures function return the protected fetcher values or read directly from the FW is no fresh values from the thread
  • N-Est values are exposed only on FW ver > 1.5.0.0

TBD - UT should be added in the future.

@Nir-Az
Nir-Az requested a review from maloel October 18, 2020 20:16
Comment thread src/l500/l500-private.h Outdated
Comment thread src/l500/l500-private.h
Comment thread src/l500/l500-device.h Outdated
Comment thread src/l500/l500-device.h Outdated
Comment thread src/l500/l500-device.cpp Outdated
Comment thread src/l500/l500-private.h
Comment thread src/l500/l500-device.cpp Outdated
Comment thread src/l500/l500-device.cpp Outdated
Comment thread src/l500/l500-device.cpp Outdated
Comment thread src/l500/l500-device.cpp Outdated
Comment thread src/l500/l500-device.cpp Outdated
Comment thread src/l500/l500-device.h Outdated
Comment thread src/l500/l500-device.h Outdated
Comment thread src/l500/l500-device.cpp Outdated
Comment thread src/l500/l500-device.cpp Outdated
Comment thread src/l500/l500-device.cpp Outdated
Comment thread src/l500/l500-device.cpp
Comment thread src/l500/l500-device.cpp Outdated
@maloel
maloel merged commit ce3e3f4 into realsenseai:development Oct 21, 2020
@Nir-Az
Nir-Az deleted the add_get_temp_thread branch December 23, 2020 07:24
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.

2 participants