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

docs(README,images): Pipeline architecture and diagrams / dataflow #455

Conversation

MilagrosMarin
Copy link
Contributor

@MilagrosMarin MilagrosMarin commented Dec 2, 2024

This PR includes updates addressing issue #77 in the aeon-docs repo - from the list in issue#60.

@MilagrosMarin MilagrosMarin changed the title docs(README,images): Pipeline architecture and diagrams / dataflow [WIP] docs(README,images): Pipeline architecture and diagrams / dataflow Dec 2, 2024
@MilagrosMarin MilagrosMarin changed the title [WIP] docs(README,images): Pipeline architecture and diagrams / dataflow docs(README,images): Pipeline architecture and diagrams / dataflow Dec 3, 2024
@MilagrosMarin
Copy link
Contributor Author

Hi @thinh, I’ve implemented the changes we discussed in the internal meeting. The PR is now ready for a second round of review. Thank you!


10. `WheelState` - wheel states (threshold, d1, delta) associated with a given `ExperimentFoodPatch`
+ `tracking.SLEAPTracking` - position tracking for object(s), from a particular `VideoSource` per chunk
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is good.
Please expand more on the 2 part tables

  • PoseIdentity - identified Subject (i.e. Identity) and stores the name of the body part used as "anchor_part"
  • Part - inferred x,y position over time for all body parts from SLEAP model

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great!
Please remove all of the part tables (Attribute, RemovalTime)
Please add Epoch in here as well

@MilagrosMarin
Copy link
Contributor Author

@ttngu207 thanks for your suggestions. I've just added them.

@ttngu207 ttngu207 merged commit 6ed933a into SainsburyWellcomeCentre:datajoint_pipeline Dec 3, 2024
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.

2 participants