Skip to content

test: rust daemon integration suite#720

Open
CougarTasker wants to merge 1 commit into
feat/rust-daemon-integrationfrom
test/rust-daemon-tests
Open

test: rust daemon integration suite#720
CougarTasker wants to merge 1 commit into
feat/rust-daemon-integrationfrom
test/rust-daemon-tests

Conversation

@CougarTasker

@CougarTasker CougarTasker commented Jun 17, 2026

Copy link
Copy Markdown
Member

Features

  • Makes the data_daemon integration harness runtime-aware so it exercises both the legacy Python daemon and the new Rust daemon.
  • Replaces the separate staging/production workflows with a single integration-platform workflow running a daemon: [python, rust] matrix.
  • Adds the local integration and performance test-runner scripts.

Bugfixes

  • None

Items

Related PRs

@CougarTasker CougarTasker self-assigned this Jun 17, 2026
@CougarTasker CougarTasker added the version:none tests, docs, CI only; no functional changes label Jun 17, 2026
@CougarTasker CougarTasker force-pushed the test/rust-daemon-tests branch from aefbb64 to 8251f1e Compare June 17, 2026 23:45
@CougarTasker CougarTasker force-pushed the feat/rust-daemon-integration branch from ac12415 to 733fc8f Compare June 17, 2026 23:54
@CougarTasker CougarTasker force-pushed the test/rust-daemon-tests branch from 8251f1e to b6c9064 Compare June 17, 2026 23:54
@CougarTasker CougarTasker changed the title test: rust/python daemon integration suite and CI matrix test: rust daemon integration suite Jun 17, 2026
@CougarTasker CougarTasker force-pushed the feat/rust-daemon-integration branch from 733fc8f to d306696 Compare June 18, 2026 00:19
@CougarTasker CougarTasker force-pushed the test/rust-daemon-tests branch from b6c9064 to cec999b Compare June 18, 2026 00:19
@CougarTasker CougarTasker force-pushed the feat/rust-daemon-integration branch from d306696 to 4eef1da Compare June 18, 2026 00:23
@CougarTasker CougarTasker force-pushed the test/rust-daemon-tests branch from cec999b to 54e6c57 Compare June 18, 2026 00:23
@CougarTasker CougarTasker force-pushed the feat/rust-daemon-integration branch from 4eef1da to 51b4e8e Compare June 18, 2026 00:24
@CougarTasker CougarTasker force-pushed the test/rust-daemon-tests branch from 54e6c57 to 2e07f3c Compare June 18, 2026 00:24
@CougarTasker CougarTasker force-pushed the feat/rust-daemon-integration branch from 51b4e8e to 04f8f1b Compare June 18, 2026 17:34
@CougarTasker CougarTasker force-pushed the test/rust-daemon-tests branch from 2e07f3c to 549c5b2 Compare June 18, 2026 17:35
@CougarTasker CougarTasker marked this pull request as ready for review June 18, 2026 17:55
@CougarTasker CougarTasker force-pushed the feat/rust-daemon-integration branch from 04f8f1b to 56da482 Compare June 18, 2026 17:59
@CougarTasker CougarTasker force-pushed the test/rust-daemon-tests branch from 549c5b2 to 1b637a4 Compare June 18, 2026 17:59
@CougarTasker CougarTasker force-pushed the feat/rust-daemon-integration branch from 56da482 to 2cde023 Compare June 19, 2026 07:31
@CougarTasker CougarTasker force-pushed the test/rust-daemon-tests branch 2 times, most recently from b57cd12 to 0e56eb4 Compare June 25, 2026 00:20
@CougarTasker CougarTasker force-pushed the feat/rust-daemon-integration branch from 2cde023 to ead7362 Compare June 25, 2026 00:20
TIMESTAMP_MODE_REAL = "real"
TIMESTAMP_MODE_STOCHASTIC = "stochastic"
STOCHASTIC_JITTER_S = 0.05
STOCHASTIC_JITTER_S = 0.004 # 120fps is 8.3 ms per frame jitter must be less than half

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.

should this be a ratio of the fps instead?

"""
with scoped_daemon_storage_env(), scoped_offline_profile():
try:
stop_daemon()

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.

Is this intentional? We should have parity between offline and online variants.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I will revert, I think its a leftover from debugging

@damonhayhurst-neuraco damonhayhurst-neuraco 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.

LGTM

@CougarTasker CougarTasker force-pushed the feat/rust-daemon-integration branch from ead7362 to 4014df2 Compare June 26, 2026 16:20
@CougarTasker CougarTasker force-pushed the test/rust-daemon-tests branch from 0e56eb4 to e7431bc Compare June 26, 2026 16:21
@CougarTasker CougarTasker force-pushed the feat/rust-daemon-integration branch from 4014df2 to 7f3e237 Compare June 26, 2026 16:40
@CougarTasker CougarTasker force-pushed the test/rust-daemon-tests branch from e7431bc to 0448e5b Compare June 26, 2026 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

version:none tests, docs, CI only; no functional changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants