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.

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

Disable the indefinite WARNING_INSUFFICIENT_TIMESYNCS wait state of the parser when started with command line args #115

Closed
2 tasks done
produceconsumerobot opened this issue May 4, 2022 · 1 comment · Fixed by #122
Closed
2 tasks done

Disable the indefinite WARNING_INSUFFICIENT_TIMESYNCS wait state of the parser when started with command line args #115

produceconsumerobot opened this issue May 4, 2022 · 1 comment · Fixed by #122

Comments

@produceconsumerobot
Copy link
Collaborator

produceconsumerobot commented May 4, 2022 •
edited

Ideal behavior when command line args are passed is to print the warning and to console and finish the program

Testing

  • Run from double click on file that has <2 timesyncs: should show warning and idle
  • Run same file from cmd line: should show warning in cmd window and exit
@produceconsumerobot produceconsumerobot mentioned this issue May 11, 2022
Merged
produceconsumerobot added a commit that referenced this issue May 11, 2022
@produceconsumerobot
Removed timesync hang from cmd line. Closes #115.
3f86ef5
@nitin710
Copy link
Collaborator

Tested working.
Parsed 2 files with insufficient timestamps user the DataParserScript.
Both files were parsed without an blocks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improved timesync map algorithm. Removed cmd line timesync warning hang. EmotiBit/ofxEmotiBit
2 participants
@produceconsumerobot @nitin710