Skip to content

Make FSGUI compatible with newer Trodes network protocol - #1

Open
XuanMabard wants to merge 2 commits into
mainfrom
trodes-network-compatible
Open

XuanMabard wants to merge 2 commits into
mainfrom
trodes-network-compatible

Conversation

@XuanMabard

@XuanMabard XuanMabard commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Update spikegadgets action, source, and config handling for compatibility with a newer version of the Trodes network protocol.
  • Adjust pulse/shortcut/action logic in fsgui/spikegadgets/action/ accordingly.

Test plan

  • Verify FSGUI connects to and streams from the updated Trodes network client
  • Exercise pulse/shortcut actions end-to-end against Trodes

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@XuanMabard
XuanMabard requested a review from shijiegu August 31, 2026 20:36
@shijiegu

shijiegu commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Is it possible to remove PR #2 commits from this PR or vice versa? There seems to be duplicated file changes across the 2 PRs. I think you can check out this branch and go back in git history and recommit only select needed files. From what I see, only namespace/import changes are part of this PR. Is that right?

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