Skip to content

Add keep-connected ros-z control for Twix#2637

Closed
schmidma wants to merge 4 commits into
HULKs:mainfrom
schmidma:twix-ros-z-keep-connected
Closed

Add keep-connected ros-z control for Twix#2637
schmidma wants to merge 4 commits into
HULKs:mainfrom
schmidma:twix-ros-z-keep-connected

Conversation

@schmidma

@schmidma schmidma commented Jun 17, 2026

Copy link
Copy Markdown
Member

Depends on #2635; GitHub diff includes earlier commits until that base merges.

Summary

  • Add keep-connected connection state to the retained-state Twix backend.
  • Make retained subscriptions and topic catalog connection-state aware.
  • Preserve retained handles during retention-only rebuilds while clearing stale diagnostics on retargets.

Test Plan

  • cargo nextest run -p twix
  • cargo check -p twix
  • cargo clippy -p twix --all-targets --locked -- -D warnings
  • git diff --check

schmidma added 4 commits June 24, 2026 15:58
Track graph mutations with local revisions, expose safe copied-revision change subscriptions, and carry revisions into graph snapshots.

Rework graph-dependent waits and dynamic schema discovery to use revision-driven waits with shared deadline handling. Document local-observation semantics and remove the configurable initial graph query path.
Replace retained debug event draining with live update receivers. Keep status-derived diagnostics on subscription status snapshots and keep the subscription lifecycle owned by SubscriptionState.
Add a retained-state ros-z backend for Twix and render Text, Plot, and Enum Plot panels from ros-z-debug retained handles.

Load legacy WebSocket panels as unsupported placeholders while preserving saved JSON, and adapt Twix to the current ros-z-debug live update API without compatibility shims.
@schmidma schmidma force-pushed the twix-ros-z-keep-connected branch from 7418636 to 6c68baf Compare June 24, 2026 20:56
@schmidma schmidma closed this Jun 29, 2026
@paudar paudar moved this from In Progress to Done in Development Jun 29, 2026
@schmidma schmidma deleted the twix-ros-z-keep-connected branch June 30, 2026 01:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants