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

Add operators for monitoring Harp device synchronization status #160

Merged
merged 2 commits into from
Oct 16, 2023

Conversation

glopesdev
Copy link
Contributor

Online monitoring of Harp device synchronization status is an important quality control test to ensure all data is correctly timestamped. This PR adds two new operators to make it easier to compute statistics for all hardware synchronized devices.

Specifically:

HeartbeatMonitor: lifts the sequence of heartbeats of a device into a named group. Each hardware synchronized device should be placed under a HeartbeatMonitor and all the sequences merged and passed into SynchronizerMonitor.

SynchronizerMonitor: receives the sequence of named device heartbeats and computes synchronization statistics aligned on the main synchronizer heartbeat.

  • MeanTimestamp: the mean timestamp value in seconds, computed from each connected Harp device
  • MeanUtcTimestamp: the epoch adjusted mean timestamp value, in UTC
  • ExpectedDeviceCount: the expected number of devices that should be reporting timestamps
  • DeviceCount: the actual number of devices that reported a timestamp in the heartbeat window
  • MaxDifference: the maximum timestamp difference, in seconds, between device timestamps
  • Elements: the names of the devices which reported timestamps in the heartbeat window

@glopesdev glopesdev added the feature New planned feature label Oct 16, 2023
@glopesdev glopesdev requested a review from aspaNeuro October 16, 2023 21:31
@glopesdev glopesdev merged commit b1ce707 into main Oct 16, 2023
1 check passed
@glopesdev glopesdev deleted the aeon-acquisition branch October 16, 2023 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New planned feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant