neuropoly / intranet.neuro.polymtl.ca

NeuroPoly's lab manual
https://intranet.neuro.polymtl.ca
4 stars 6 forks source link

Update data curation #108

Open jcohenadad opened 7 months ago

jcohenadad commented 7 months ago

https://github.com/neuropoly/data-management/issues/282

NathanMolinier commented 6 months ago

I updated our intranet with our new convention. Can I have some quick feedback ? @jcohenadad @valosekj

I will then start updating our datasets !

NathanMolinier commented 6 months ago

I actually have one last point here https://github.com/neuropoly/data-management/issues/282#issuecomment-1889642614

jcohenadad commented 6 months ago

@NathanMolinier next time can you please do the modifs within a PR-- otherwise it is difficult to see all the changes pertaining to the convention we agreed on

I understand these two commits are to be reviewed:

jcohenadad commented 6 months ago

Can I have some quick feedback ? @jcohenadad @valosekj

done!

NathanMolinier commented 6 months ago

Thank you ! I will create a new PR for the future changes

NathanMolinier commented 6 months ago

I updated the convention about how we want to handle derivatives folders.

Is it ok ? Here is the associated PR

jcohenadad commented 6 months ago

@NathanMolinier can you pls add an example of JSON with manual label creation here

to clarify, we should create another example ONLY with manual correction. This one below is lost

image