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

Support linking to multiple camera devices #5

Closed
wants to merge 13 commits into from
Closed

Conversation

rly
Copy link
Owner

@rly rly commented Jan 26, 2022

Pending a fix for hdmf-dev/hdmf#693

@rly
Copy link
Owner Author

rly commented Feb 14, 2023

The original and labeled video files (currently represented as a list but my be changed to a set from #12) is ordered. We intend the set of cameras to align with these lists, so the nth video was generated by the nth camera. However, a set of N links (quantity: '*') is unordered. There is no simple and straightforward way to align these in the PoseEstimation data type.

The proposal in #12 would change the video files to be ImageSeries objects, which already have an optional link to a Device that would represent the camera. That should suffice. So the devices field should be removed from the PoseEstimation type in the resolution of #12 and this PR will no longer be necessary.

@rly rly mentioned this pull request Nov 22, 2023
7 tasks
@rly rly mentioned this pull request Apr 12, 2024
@rly
Copy link
Owner Author

rly commented Jul 18, 2024

Already implemented in a different PR.

@rly rly closed this Jul 18, 2024
@rly rly deleted the fix/multi_device branch July 18, 2024 23:48
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.

1 participant