Skip to content

feature: Better stream plotter#122

Merged
gilbert-sci merged 8 commits intomainfrom
gilbert/different-stream-plot
Jun 18, 2025
Merged

feature: Better stream plotter#122
gilbert-sci merged 8 commits intomainfrom
gilbert/different-stream-plot

Conversation

@gilbert-sci
Copy link
Copy Markdown
Contributor

@gilbert-sci gilbert-sci commented Jun 9, 2025

Summary

Updates the stream plotter to use taps instead of LibNDTP. Also updates the plotting view to be a bit more performant.

Since it seems like everyone uses the previous plotter I made when I first started as a quick "Is this thing running and does the signal look approximately correct".

Testing

  • Tested locally with a braodband
# List things you can listen to
python3 synapse/cli/__main__.py -u "<device name>" read <path to config> --list-taps

# Listen to a tap that is streaming BroadbandFrame
python3 synapse/cli/__main__.py -u "<device name>" read <path to config> --tap-name broadband_source_1

# Plot
python3 synapse/cli/__main__.py -u "<device name>" read <path to config> --tap-name broadband_source_1 --plot

# Save to disk as hdf5
python3 synapse/cli/__main__.py -u "<device name>" read <path to config> --tap-name broadband_source_1 --output <directory>

@gilbert-sci gilbert-sci self-assigned this Jun 9, 2025
@gilbert-sci gilbert-sci marked this pull request as draft June 9, 2025 22:05
Base automatically changed from gilbert/hdf5-reader to main June 11, 2025 22:25
@gilbert-sci gilbert-sci force-pushed the gilbert/different-stream-plot branch from f1a8a2b to 77b7460 Compare June 13, 2025 22:19
@gilbert-sci gilbert-sci marked this pull request as ready for review June 13, 2025 22:23
@slagernate
Copy link
Copy Markdown
Contributor

slagernate commented Jun 13, 2025

maybe a stretch but does this handle TTL channels? 👉 👈 (the difference being that TTL should be plotted as a digital signal instead of an analog signal)

@gilbert-sci
Copy link
Copy Markdown
Contributor Author

@slagernate not yet but it is now on my radar! I would like to get this one done first, then i can add it :)

@gilbert-sci
Copy link
Copy Markdown
Contributor Author

Example
image

@gilbert-sci
Copy link
Copy Markdown
Contributor Author

@slagernate @namthor9 @emmazhou bump

Copy link
Copy Markdown
Contributor

@namthor9 namthor9 left a comment

Choose a reason for hiding this comment

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

hell yeah

@gilbert-sci gilbert-sci merged commit 5a0d199 into main Jun 18, 2025
1 check passed
@gilbert-sci gilbert-sci deleted the gilbert/different-stream-plot branch June 18, 2025 22:37
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