lightsources / BES-XPCS-Pilot

Work with BES Data Science Team to develop Python code to write NeXus data file with proposed NeXus definitions.
0 stars 0 forks source link

generate a list of required that should be optional and vice versa #60

Closed ambarb closed 3 years ago

ambarb commented 3 years ago

not required using Pete's NXxpcs mock webpage

If required means the field is required, but the field doesn't require a value, then maybe this is not essential.

ambarb commented 3 years ago

@prjemian I am happy to contribute to PR if we need to addres it. Can you answer my question about the meaning of "required"

prjemian commented 3 years ago

required means the field must be present and a value of the expected data type must be provided

We can make such fields optional if a value is not always expected to be available. (not truly required)

ambarb commented 3 years ago

@ambarb to make sure the list is complete.

ambarb commented 3 years ago

https://github.com/lightsources/BES-XPCS-Pilot/blob/562404d66915d5bd3e1194329be176b0756d4592/NeXus/NXxpcs.nxdl.xml#L173 is how to make things not required (required by default)

ambarb commented 3 years ago

we all believe that we are done (with pushes)

if there are more, they will come up in our XPCS community review.