Skip to content

Add sourced console support - #13

Draft
ataffanel wants to merge 1 commit into
mainfrom
sourced-console
Draft

Add sourced console support#13
ataffanel wants to merge 1 commit into
mainfrom
sourced-console

Conversation

@ataffanel

Copy link
Copy Markdown
Member

Summary

  • add sourced-console catalog discovery with human and CSV output
  • stream one source by exact catalog path with replayed history and live output
  • preserve the existing legacy console UX and legacy-only local history behavior
  • disable an enabled source with bounded cleanup before disconnecting
  • document the initial single-source UX and add focused unit coverage

Related work

Verification

  • cargo test — 12 tests passed
  • cargo build passed without warnings
  • git diff --check passed
  • hardware acceptance passed on radio://0/22/2M/E7E7E7E7E7 with deck:bcCam
    • catalog discovery in human and CSV modes
    • formatted replay/live streaming
    • raw replay/live streaming, including a trailing partial line at timeout
    • clean disable and re-enable across consecutive runs
    • missing source exits 20 and reports the available path
    • legacy console regression check

Strict repository-wide cargo fmt --all -- --check is currently blocked by extensive pre-existing formatting differences. Strict cargo clippy --all-targets -- -D warnings is blocked by 39 pre-existing findings in unrelated code; no finding concerns this implementation.

Before merge

  • Replace the pinned crazyflie-lib git revision with the released crates.io version containing sourced-console support.

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.

1 participant