Fix nwb_version so that dandi can read it

AllenInstitute / MIES

Multichannel Igor Electrophysiology Suite

Other

22 stars 6 forks source link

Closed t-b closed 2 weeks ago

t-b commented 4 weeks ago

[x] https://github.com/AllenInstitute/IPNWB/pull/46
[x] Integrate dandi into CI pipeline, dandi.exe validate --ignore '(NWBI|DANDI)' XXX
[x] Cleanup commits

Close #2243 Close #1988

t-b commented 3 weeks ago

@nrsc Can you try with this version of MIES? It should create files which pass dandi validate in this regard. You can grab the installer, BuildInstaller-user-assets, from https://github.com/AllenInstitute/MIES/actions/runs/10738308861?pr=2254

t-b commented 2 weeks ago

I'm merging now as we have CI which checks for the issue so manual testing by @nrsc should not be necessary.

nrsc commented 2 weeks ago

Hi @t-b, apologies for the time away. I'm going to pull the executable and reinstall, then run the export to test the validation. Cheers

t-b commented 2 weeks ago

Hi @t-b, apologies for the time away. I'm going to pull the executable and reinstall, then run the export to test the validation. Cheers

No worries.