Skip to content

Fix set-gain-stress-test for D555 - #15306

Merged
OhadMeir merged 2 commits into
realsenseai:developmentfrom
OhadMeir:unit_tests
Jun 29, 2026
Merged

OhadMeir merged 2 commits into
realsenseai:developmentfrom
OhadMeir:unit_tests

Conversation

@OhadMeir

Copy link
Copy Markdown
Contributor

Tracked on [RSDEV-12382]

Copilot AI review requested due to automatic review settings June 29, 2026 06:50

Copilot AI left a comment

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.

Pull request overview

This PR updates a weekly live unit-test that repeatedly sets the gain option so it behaves reliably on D555 when connected via DDS, where option-set broadcasts can race with immediate subsequent option sets.

Changes:

  • Detect DDS-connected devices during the test run.
  • Add a short delay between consecutive gain sets on DDS devices to avoid broadcast/reply races.

@OhadMeir
OhadMeir requested a review from Nir-Az June 29, 2026 07:11
Copilot AI review requested due to automatic review settings June 29, 2026 08:46

@Nir-Az Nir-Az left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

@OhadMeir

Copy link
Copy Markdown
Contributor Author

CI running this weekly test https://rsjenkins.realsenseai.com/job/LRS_libci_pipeline/17012/

@OhadMeir
OhadMeir merged commit 8f2cc25 into realsenseai:development Jun 29, 2026
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants