Skip to content
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

Y16 calibration format test #12234

Merged

Conversation

Tamir91
Copy link
Contributor

@Tamir91 Tamir91 commented Sep 28, 2023

Tracked on [RSDEV-724]

Porting an existing D400 test from Cpp to Python.

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.

few comments


profile_y16 = next(p for p in depth_sensor.profiles if p.format() == rs.format.y16)
test.check(profile_y16)
log.d(str(profile_y16))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed

@remibettan remibettan self-requested a review September 28, 2023 12:43
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

y16_streamed = True


timer = Timer(30)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please reduce to 5.
If we don't get a frame in 5 seconds, we have another problem :)
No need to stall the LibCI on that case

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You right, we confused with this value too.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done, a value is 5 now.

@Nir-Az
Copy link
Collaborator

Nir-Az commented Oct 1, 2023

Let's also delete the old cpp test file, no need anymore right?

@Nir-Az
Copy link
Collaborator

Nir-Az commented Oct 2, 2023

image

Please remove

@Tamir91 Tamir91 force-pushed the Y16-calibration-format-test branch from 0aa3d80 to 6a90b19 Compare October 2, 2023 08:59
Copy link
Collaborator

@Nir-Az Nir-Az left a comment

Choose a reason for hiding this comment

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

LGTM

@Nir-Az Nir-Az merged commit 8809af3 into IntelRealSense:development Oct 2, 2023
15 of 16 checks passed
@Tamir91 Tamir91 deleted the Y16-calibration-format-test branch October 12, 2023 13:13
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