Skip to content

Combined p1_capture, p1_filter, and p1_print utilities.#402

Merged
adamshapiro0 merged 79 commits intomasterfrom
p1-capture
Apr 23, 2026
Merged

Combined p1_capture, p1_filter, and p1_print utilities.#402
adamshapiro0 merged 79 commits intomasterfrom
p1-capture

Conversation

@adamshapiro0
Copy link
Copy Markdown
Collaborator

@adamshapiro0 adamshapiro0 commented Apr 22, 2026

Changes

  • Refactored p1_capture to combine capabilities with p1_filter and p1_print into a single tool
  • Added message type filter invert option to MixedLogReader and FileIndex

Fixes

  • Fixed FileIndex time range filtering if the end time is earlier than the start of the data and the first messages do not have P1 timestamps
  • Expand ~/ when parsing input/output file paths
  • Fixed Python 3.8/3.9 compatibility issues

@adamshapiro0 adamshapiro0 requested a review from bhers4 April 22, 2026 19:17
@adamshapiro0 adamshapiro0 self-assigned this Apr 22, 2026
Comment thread python/fusion_engine_client/applications/p1_capture.py Fixed
Comment thread python/fusion_engine_client/applications/p1_capture.py Fixed
Comment thread python/tests/test_p1_capture.py Fixed
Comment thread python/fusion_engine_client/applications/p1_capture.py Fixed
Comment thread python/fusion_engine_client/messages/measurements.py Fixed
@adamshapiro0 adamshapiro0 force-pushed the p1-capture branch 2 times, most recently from 1e1ebba to d8bb412 Compare April 22, 2026 19:25
@adamshapiro0 adamshapiro0 changed the base branch from master to wrapped April 22, 2026 19:28
@adamshapiro0 adamshapiro0 changed the title Combined p1_capture, p1_filter, and p1_print` utilities. Combined p1_capture, p1_filter, and p1_print utilities. Apr 22, 2026
Copy link
Copy Markdown
Contributor

@bhers4 bhers4 left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Comment thread python/fusion_engine_client/applications/p1_capture.py Outdated
Comment thread python/fusion_engine_client/applications/p1_capture.py Outdated
Comment thread python/fusion_engine_client/applications/p1_capture.py Outdated
Comment thread python/fusion_engine_client/applications/p1_capture.py Outdated
Comment thread python/fusion_engine_client/parsers/file_index.py
Comment thread python/fusion_engine_client/applications/p1_capture.py Outdated
Comment thread python/fusion_engine_client/applications/p1_capture.py
Comment thread python/fusion_engine_client/applications/p1_capture.py
Comment thread python/fusion_engine_client/applications/p1_capture.py
Base automatically changed from wrapped to master April 23, 2026 22:56
Comment thread python/fusion_engine_client/messages/signal_def_gen.py Fixed
@adamshapiro0 adamshapiro0 merged commit ce30844 into master Apr 23, 2026
23 checks passed
@adamshapiro0 adamshapiro0 deleted the p1-capture branch April 23, 2026 23:54
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.

2 participants