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

Clocktools sample trace #426

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

ZhuokunDing
Copy link
Contributor

  1. Reorganize fetch_timing_data into:
    • scan_image_correction: fetch delay for the entire field or a single unit based on the argument scan_key
    • fetch_timing_data: fetch the corresponding timestamps based on the argument timing_type
    • interpolate_timing_data: interpolates the timestamps from the source timestamps to the target time stamps and vice versa.
  2. Add time-stimulus as an option in interpolating_timing_data to load flip times in stimulus clock. The current implementation concatenates the flip times found in stimulus.Trial, which is not necessary since concatenated flip times already exist in stimulus.Sync.Syncmodel. However Syncmodel tables are not populated for the platinum scans yet. The current implementation should be updated after the Syncmodel tables are repopulated for the platinum scans.
  3. Implement sample_trace using fetch_timing_data, interpolate_timing_data and interpolating_signal_data to sample the traces at arbitrary time points in stimulus time. This function could be extended to sample with behavior time as well, not implemented at the moment.

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

Successfully merging this pull request may close these issues.

Missing feature of Clocktool: Sample response/behavior traces at arbitrary behavior/stimulus times
1 participant