Skip to content

Update websocket for changed signal structure #58

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
katelyn-chun wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Update websocket for changed signal structure #58

katelyn-chun wants to merge 6 commits into from

Conversation

@katelyn-chun
Copy link
Contributor

@katelyn-chun katelyn-chun commented Jan 25, 2026

What?

  • Updated websocket to normalize new data format

How?

  • Mapped the batches in websocket instead of signal-graph-node, for the new 2D array format
  • Ensured the entries were in the format of time, signal1, signal2, signal3, signal4
  • Changed server.js to send sample data with the new batched format

@MaddieWright MaddieWright self-requested a review February 1, 2026 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MaddieWright MaddieWright MaddieWright requested changes

@HeisSteve HeisSteve Awaiting requested review from HeisSteve

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@katelyn-chun @MaddieWright