You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
ATM it must be in years (according to schema and hardcoded there assumptions), although text suggests differently
If no `units` is provided for age, it will be assumed to be in years relative to date of birth.
which was suggested/accepted in code review of #459 but 1. no alternative units are specified (thus unknown/useless) ; 2. such relaxation was not later reflected in the schema which now just hardcode via unit: years without alternative unit values provided. (not even sure if we formalize units to be machine readable ATM, e.g. see #1631 to fix some values)
@bids-standard/bep032 is working on animal ecephys. In "animal world" restriction to age in years is often too restrictive, ambiguous in conversion (# of days in a year changes, # of days in month variable, etc) or not even expressive enough (pre-gestational age etc; let's keep that for a separate issue). So we need to improve on specification of age by allowing other "units" and also allow for metadata field following some dictionary which would allow specification of the "age reference". Here is what is done in other standards/uses:
NWB uses "age" of "text" type . For DANDI then it allows for treating ISO 8601 Durations values if those are specified, enforced during metadata validation using nwbinspector . Use of ISO 8601 makes any duration inambigous.
Since we are "unit" oriented ATM, I think the easiest resolution for this aspect would be to just do implement in the schema on what is promised in the text:
Consider "Units" sidecar file value
Define explicit "unit" values which would be accepted for age: year, month, day, hour, minute, s, ms
Question to schema folks (@effigies@rwblair@nellh): how difficult would be to generalize current check
ATM it must be in years (according to schema and hardcoded there assumptions), although text suggests differently
which was suggested/accepted in code review of #459 but 1. no alternative units are specified (thus unknown/useless) ; 2. such relaxation was not later reflected in the schema which now just hardcode via
unit: years
without alternative unit values provided. (not even sure if we formalize units to be machine readable ATM, e.g. see #1631 to fix some values)@bids-standard/bep032 is working on animal ecephys. In "animal world" restriction to age in years is often too restrictive, ambiguous in conversion (# of days in a year changes, # of days in month variable, etc) or not even expressive enough (pre-gestational age etc; let's keep that for a separate issue). So we need to improve on specification of age by allowing other "units" and also allow for metadata field following some dictionary which would allow specification of the "age reference". Here is what is done in other standards/uses:
Since we are "unit" oriented ATM, I think the easiest resolution for this aspect would be to just do implement in the schema on what is promised in the text:
Question to schema folks (@effigies @rwblair @nellh): how difficult would be to generalize current check
to support use of the "Units" in the side car?
The text was updated successfully, but these errors were encountered: