catalystneuro / vame-desktop

A desktop application for the Variational Animal Motion Encoding project.
GNU General Public License v3.0
1 stars 0 forks source link

feat: new description on pose_ref_index #23

Closed nathan-vm closed 2 weeks ago

nathan-vm commented 2 weeks ago
garrettmflynn commented 2 weeks ago

@katiekly Can you confirm these are the same across all data that will be loaded into VAME?

garrettmflynn commented 2 weeks ago

@nathan-vm Thanks for covering this! I forget if I've written in the min/max logic into the number inputs.

Can you confirm and add if this is missing in the DynamicForm?

nathan-vm commented 2 weeks ago

@nathan-vm Thanks for covering this! I forget if I've written in the min/max logic into the number inputs.

Can you confirm and add if this is missing in the DynamicForm?

@garrettmflynn It is not covered but i can manage do tha in this PR allready

I can handle this recursively or simply copy the base inputs to another file and have a check in the DynamicForm:

 isArray? <ArrayItemWrapper> {.map(i => <BaseInput/>)} </ArrayItemWrapper> : <BaseInput/>
katiekly commented 2 weeks ago

Hi @garrettmflynn, yes all body parts should be the same for each CSV loaded.

garrettmflynn commented 2 weeks ago

@katiekly Sorry what I meant is are these column headers of the CSV files the same across all Projects? Just want to confirm that specific property of the CSV files.

garrettmflynn commented 2 weeks ago

@nathan-vm Whichever you think is best! Seems like it might be more appropriate to break the input generation out to a component of its own

katiekly commented 2 weeks ago

@garrettmflynn, thanks for clarifying. No, the CSVs column header may vary between Projects. Some projects would require different annotation points, which defines the CSV column headers.

garrettmflynn commented 2 weeks ago

@nathan-vm So Katie's last comment means that this cannot just be a description in the json file, but would have to be dynamically loaded into the project and injected into the schema. Still much easier than I was originally thinking!

Also not sure about your availability, but we're meeting in an hour if you'd like to join. I'll introduce you to the client.

nathan-vm commented 2 weeks ago

@katiekly I think what I did here work just fine, @garrettmflynn how can I send to her for testing it before I merge it ?

garrettmflynn commented 2 weeks ago

Katie should be able to pull this branch and try it out!