Skip to content
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.

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

Major Refactor of Networking Widget #971

Closed
8 tasks
retiutut opened this issue May 7, 2021 · 4 comments
Closed
8 tasks

Major Refactor of Networking Widget #971

retiutut opened this issue May 7, 2021 · 4 comments

Comments

@retiutut
Copy link
Member

retiutut commented May 7, 2021

Problem

W_Networking.pde is a "hot mess".

Expected

  • Remove unnecessary code
  • Reconfirm all data outputs for all data modes or improve as needed

Operating System and Version

All OS

GUI Version

v5.0.4

Running standalone app

App and Dev

Type of OpenBCI Board

All boards

Are you using a WiFi Shield?

All boards

Console Log

It's great that the networking widget prints information to console when new stream is started. Keep doing this!


  • For certain data types, Networking widget does not stream data unless the corresponding widget is open. GUI 5.0.4 streams out 0's or null instead. Begin to look into this. Make a new ticket if it seems reasonably possible to compute all data in the background even if some widgets aren't open. Target widget must be open in order to stream from networking widget #1094

  • Remove unnecessary code

  • Reconfirm all data outputs for all data modes or improve as needed

  • LSL - Look at all instances of push_sample and push_chunk to see what is correct and proper, and make changes so that this networking mode is refined and can operate as the "most-used" protocol.

  • Double check networking output guide.... mention that data may be sent in "chunks"for LSL and maybe other data modes. (e.g. [[sample1 s1 s1 s1],[sample2 s2 s2 s2],[sample3 etc.]] )

  • Confirm and review Serial

  • Confirm and review UDP

  • Confirm and review OSC

  • Confirm and review LSL

@retiutut retiutut self-assigned this May 7, 2021
@retiutut retiutut added this to the 5.1.0 milestone May 7, 2021
@retiutut retiutut modified the milestones: 5.1.0, 5.0.7 Aug 11, 2021
@retiutut
Copy link
Member Author

Another user has noticed some possible quirky behavior in #989

@retiutut
Copy link
Member Author

Relates heavily to #1042

@retiutut
Copy link
Member Author

Workable fix exists using NeuroFly project GUI. Port fixes to networking widget there to the public GUI.

@retiutut
Copy link
Member Author

For certain data types, Networking widget does not stream data unless the corresponding widget is open. GUI 5.0.4 streams out 0's or null instead. Begin to look into this. Make a new ticket if it seems reasonably possible to compute all data in the background even if some widgets aren't open. #1094

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants