Skip to content

conftest: settle 4s after enabling D585-family devices (FW accel bring-up window) - #15439

Merged
Nir-Az merged 3 commits into
realsenseai:developmentfrom
Nir-Az:d585-bringup-settle
Jul 21, 2026
Merged

Nir-Az merged 3 commits into
realsenseai:developmentfrom
Nir-Az:d585-bringup-settle

Conversation

@Nir-Az

@Nir-Az Nir-Az commented Jul 16, 2026 •

Copy link
Copy Markdown
Collaborator

Overview: Test harness now waits 4 seconds after powering on a D585 Prototype device before tests start streaming, working around a FW window in which the accelerometer delivers no data.

Tracked on [RSDEV-13011]

Why

D585 Prototype FW produces no accelerometer data if streaming starts within the first ~5-7 seconds after power-on (RSDEV-13011). The pipeline aggregator publishes framesets only when every resolved stream has delivered, so any test whose config includes the IMU fails with Frame didn't arrive within 5000. In a full nightly with the D585 Prototype enabled this failed 4 tests (rec-play test_pipeline_interface, test_pipeline_start_stop, test_accel_streaming[100/200]) and retry-masked a fifth (depth-unit).

Summary

  • module_device_setup (conftest): after the device-enable handshake, if an enabled device is a D585 Prototype ("D585 Proto" name match — D585S and production D585 are not affected and not delayed), sleep D585_BRINGUP_SETTLE_SEC = 4 (measured threshold ~3s + 1s margin) before yielding to tests. Both single- and multi-device paths.
  • No cost for other SKUs; ~4s per D585 Prototype module (+23s on a full nightly).
  • Temporary workaround — to be reverted once the FW fix for RSDEV-13011 is deployed (removal tracked in RSDSO-21766).

Test plan

  • infra-tests: 148 passed, 0 failed (rerun after the name-gate narrowing)
  • Full nightly (D585 Prototype included, D585S excluded), same bench as baseline: baseline = 4 failed + 5 retried; with this patch = 356 passed, 0 failed, 0 retried (all platforms green)
  • Settle confirmed active in per-test logs ("D585 bring-up settle: waiting 4s before tests")

🤖 Generated by AI

@sysrsbuild-gh-agentic sysrsbuild-gh-agentic 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.

rs-agentic bot automated review — 3 inline comments below.

Comment thread unit-tests/conftest.py Resolved
Comment thread unit-tests/conftest.py Resolved
Comment thread unit-tests/conftest.py Resolved
@Nir-Az
Nir-Az merged commit 7ef9ecf into realsenseai:development Jul 21, 2026
31 of 32 checks passed
@Nir-Az
Nir-Az deleted the d585-bringup-settle branch August 11, 2026 10: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.

3 participants