Refactor project structure and split vision operators into separate packages #219
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
glopesdev
added
fix
glopesdev
force-pushed
the
vision-refactor
branch
from
82481f3 to
59461af
Compare
aspaNeuro
approved these changes
Jul 17, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As functionality and adoption of the Aeon framework continues to grow, we have gotten to a point where there is too much packed into the single core
Aeon.Acquisitionpackage as discussed in #214.This PR splits the previous acquisition package into three new packages:
Aeon.Video: all video capture interfaces and video controller / triggering devicesAeon.Vision: computer vision tasks such as ROI activity, blob detection, etc.Aeon.Vision.Sleap: specialized package for SLEAP pose estimation (to isolate tensorflow and CUDA dependencies)These changes should make it easier for researchers to use the Aeon framework for simpler experiments which may not require the bulk of all dependencies.
The project structure has been refactored to adopt the improved artifacts output layout for builds and to version all components together to simplify releases.
Obsolete experimental environments and scripts have been removed as they have since moved to the SainsburyWellcomeCentre/aeon_experiments repo.
Finally, CI runs have been modernized and updated to run tests on both Windows and Linux.
Fixes #214