Skip to content

add pixel temporal noise metric #13786

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

Merged
merged 1 commit into from
Feb 25, 2025

Conversation

gilpazintel
Copy link
Contributor

add pixel temporal noise metric
tracked by DQT RSDSO-7359
update readme file

use upper case for num images

revert code which takes N frames parameter from config file. fix deprojection issue
set the depth tensor size to be as the ROI size.

change pixel noise from mm to %

change default number of images (N) from 10 to 40. start the noise calculation only once we have reached N images

  • pending definition of % is regards of pixel noise (instead of mm)

update readme file

use upper case for num images

revert code which takes N frames parameter from config file.
fix deprojection issue
set the depth tensor size to be as the ROI size.

change pixel noise from mm to %

change default number of images (N) from 10 to 40.
start the noise calculation only once we have reached N images
* pending definition of % is regards of pixel noise (instead of mm)

add pixel temporal noise metric to DQT RSDSO-7359
@gilpazintel
Copy link
Contributor Author

gilpazintel commented Feb 24, 2025

this PR replace #13750

Copy link
Contributor

@remibettan remibettan left a comment

Choose a reason for hiding this comment

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

LGTM

@gilpazintel gilpazintel changed the title apply code review changes add pixel temporal noise metric Feb 25, 2025
@Nir-Az Nir-Az merged commit a6f90c9 into IntelRealSense:development Feb 25, 2025
46 checks passed
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