scipp / chexus

Validate NeXus files
https://scipp.github.io/chexus/
BSD 3-Clause "New" or "Revised" License
0 stars 2 forks source link

Should we check that `top_dead_center` are NX_NUMBER and not NXlog #28

Open nvaytet opened 2 months ago

nvaytet commented 2 months ago

According to the nexus standard, top_dead_center in the nexus files should be NX_NUMBER. But in some files, it is found to be NXlog. Chexus does not seem to pick this up?

nvaytet commented 2 months ago

Alternatively, chexus may need to check that NXlog has a value and raise a warning if it does not? Currently, a warning is raised by scippnexus when trying to load such a log, but chexus does not catch it.

SimonHeybrock commented 1 month ago

As far as I understand NeXus, anything that is a "number" can also become a time-dependent NXlog. See for example NXtransformations, ...