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

original_videos should be a link #12

Open
bendichter opened this issue Feb 14, 2023 · 1 comment · May be fixed by #13
Open

original_videos should be a link #12

bendichter opened this issue Feb 14, 2023 · 1 comment · May be fixed by #13

Comments

@bendichter
Copy link
Collaborator

original_videos should be a list of links to ImageSeries objects

@CodyCBakerPhD
Copy link

An alternative approach to get use of this in the IBL pipeline faster and cause less immediate trouble for DLC and SLEAP would be a smoother deprecation cycle such as

  1. Introduce a new field image_series that does whatever schema magic must be done to specify it as a 'list' of links to ImageSeries
  2. Add deprecation warning to original_videos telling people to use the new field instead
  3. Once DLC and SLEAP update their usage we can safely remove it

Also for context, this to allow uploading of the videos to DANDI, which only supports external files if they are contained in an ImageSeries - the flip side of adjusting DANDI to be compatible with this field of a PoseEstimation would be much more challenging and too hyper-specific of a treatment just for an extension data type IMO

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 a pull request may close this issue.

2 participants