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

Improve online harp synchronization qc #197

Open
1 of 3 tasks
bruno-f-cruz opened this issue Jan 24, 2024 · 2 comments
Open
1 of 3 tasks

Improve online harp synchronization qc #197

bruno-f-cruz opened this issue Jan 24, 2024 · 2 comments
Assignees
Milestone

Comments

@bruno-f-cruz
Copy link
Collaborator

bruno-f-cruz commented Jan 24, 2024

  • Check heartbeat jitter (Not sure this is worth it)
  • Wait for Add hearbeat register harp-tech/protocol#45 and implement heartbeat checker (this would be the preferred strategy)
  • Given a list of expected devices check the clock generator pinout (completed in 2a7563e)
@bruno-f-cruz bruno-f-cruz self-assigned this Jan 24, 2024
@bruno-f-cruz bruno-f-cruz changed the title Implement period synchronization check Implement periodic synchronization check Jan 25, 2024
@bruno-f-cruz bruno-f-cruz added this to the 0.1.1 milestone Feb 22, 2024
@bruno-f-cruz bruno-f-cruz modified the milestones: 0.1.1, 0.2.0 Mar 6, 2024
@bruno-f-cruz
Copy link
Collaborator Author

Do low hanging fruit now. But push a better qc to later once the new synch device is ready

@bruno-f-cruz bruno-f-cruz modified the milestones: 0.4.0, 0.5.0 Jul 24, 2024
@bruno-f-cruz
Copy link
Collaborator Author

bruno-f-cruz commented Nov 11, 2024

A proof of concept for a QC routine can be seen here: AllenNeuralDynamics/Aind.Behavior.Telekinesis@7943962

edit:
Implemented in 2a7563e

@bruno-f-cruz bruno-f-cruz changed the title Implement periodic synchronization check Improve online harp synchronization qc Nov 12, 2024
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

1 participant