Replies: 2 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
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