AllenInstitute / ecephys_etl_pipelines

Pipelines and modules for processing extracellular electrophysiology data
Other
1 stars 2 forks source link

12 move and update align timestamps #17

Closed njmei closed 3 years ago

njmei commented 3 years ago

This PR is intended to address issue #12.

This PR should be reviewed AFTER #16, but should be merged BEFORE it.

Along the way of getting tests to pass and new VBN data to work properly, it was discovered that old tests were using on-prem data resources. The brunt of the work in this PR was developing more light weight test resources for both Visual Coding Neuropixels as well as new test resources for Visual Behavior Neuropixels for testing align_timestamps functionality.

njmei commented 3 years ago

Branch has been updated to address comments. Module has been run with dev LIMS and looks to be working:

image

Associated LIMS PR is here: http://stash.corp.alleninstitute.org/projects/TECH/repos/lims/pull-requests/783/overview