Replies: 3 comments
-
This pattern has some caveats that I also went through when implementing https://github.com/AllenNeuralDynamics/aind-data-schema-models/pull/61/files that I will document here.
|
Beta Was this translation helpful? Give feedback.
0 replies
-
Additional note here: there are many records where modality was uploaded as not a list, although I'm sure some people wouldn't pay attention the mis-named field almost certainly contributes to this |
Beta Was this translation helpful? Give feedback.
0 replies
-
Snuck this into 2.0, closing. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
It accepts a list, would be less confusing if it was plural.
This wouldn't be a backward compatible change, would need to add a before validator to swap the name in old metadata files.
Beta Was this translation helpful? Give feedback.
All reactions