Skip to content

Add FW errors monitor test - #14318

Merged
Nir-Az merged 4 commits into
realsenseai:developmentfrom
Nir-Az:fw-errors
Oct 8, 2025
Merged

Add FW errors monitor test#14318
Nir-Az merged 4 commits into
realsenseai:developmentfrom
Nir-Az:fw-errors

Conversation

@Nir-Az

@Nir-Az Nir-Az commented Sep 29, 2025

Copy link
Copy Markdown
Collaborator

Tracked on [RSDSO-20609]

@Nir-Az
Nir-Az requested a review from ymodlin September 29, 2025 15:12
@Nir-Az
Nir-Az requested review from OhadMeir, alexkunin-gh and Copilot and removed request for alexkunin-gh and ymodlin September 30, 2025 08:20

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 adds a new test file to monitor firmware errors during device streaming operations. The test validates hardware stability by ensuring no firmware errors occur during a streaming session.

Key changes:

  • Implements a firmware error monitoring test for D400* and D500* devices
  • Streams for 10 seconds while monitoring for hardware and firmware error notifications
  • Validates that zero firmware errors are detected during the streaming period

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread unit-tests/live/fw/test-fw-errors.py Outdated
Comment thread unit-tests/live/fw/test-fw-errors.py Outdated
@Nir-Az Nir-Az added the ai-assist AI was used to develop this feature label Sep 30, 2025
@Nir-Az

Nir-Az commented Sep 30, 2025

Copy link
Copy Markdown
Collaborator Author

Manual trigger CI passed with nightly context
image

@OhadMeir OhadMeir 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.

Can we simulate expected errors and test for it too?

@Nir-Az
Nir-Az merged commit 7ba6aea into realsenseai:development Oct 8, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-assist AI was used to develop this feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants