search

catalystneuro / IBL-to-nwb

Conversion of IBL data to NWB format.
BSD 3-Clause "New" or "Revised" License
2 stars 3 forks source link

Add API consistency checks #88

Closed CodyCBakerPhD closed 1 week ago

CodyCBakerPhD commented 1 week ago

Would replace #77

@grg2rsr The uncommented blocks here show how I imagine integrating the consistency checks between NWB and ONE as a part of the file creation process (prior to DANDI upload)

Let me know what you think

CodyCBakerPhD commented 1 week ago

@grg2rsr You've also been added as a collaborator, so if you like this PR, feel free to clone the CN repo (not the IBL fork) to do the rest of the work on 'filling in the rest' by uncommenting the blocks and adjusting them to the suggested structure

grg2rsr commented 1 week ago

should I commit my changes to this branch, or make a new one?

CodyCBakerPhD commented 1 week ago

@grg2rsr If you want to be fancy, you could branch from this branch and open a PR with this branch as its base

Whichever is easier for you though