Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect place for NXtomo value in tomo_standard.nxs internal structure #545

Open
yousefmoazzam opened this issue Feb 14, 2025 · 1 comment
Labels

Comments

@yousefmoazzam
Copy link
Collaborator

From the NXtomo application definition, the definition field should be a direct child of the NXentry. However, in tomo_standard.nxs, it looks like the definition field is typo-ed to defination and is nested further down within /entry1/tomo_entry than it should be:

[twi18192@pc0074 httomo (main)]$ h5dump -a /entry1/tomo_entry/definition tests/test_data/tomo_standard.nxs
HDF5 "tests/test_data/tomo_standard.nxs" {
ATTRIBUTE "definition" {h5dump error: unable to open attribute "definition"

}
}
[twi18192@pc0074 httomo (main)]$ h5dump -a /entry1/tomo_entry/data/defination tests/test_data/tomo_standard.nxs
HDF5 "tests/test_data/tomo_standard.nxs" {
ATTRIBUTE "defination" {
   DATATYPE  H5T_STRING {
      STRSIZE H5T_VARIABLE;
      STRPAD H5T_STR_NULLTERM;
      CSET H5T_CSET_UTF8;
      CTYPE H5T_C_S1;
   }
   DATASPACE  SCALAR
   DATA {
   (0): "NXtomo"
   }
}
}
@yousefmoazzam
Copy link
Collaborator Author

yousefmoazzam commented Feb 28, 2025

One more issue with tomo_standard.nxs is that I think the definition should be a dataset rather than an attribute, after seeing how data from beamlines with an NXtomo have the definition as a dataset, not an attribute.

For example, see the following I12 file (note the use of -d not -a in the h5dump command in order to successfully inspect it):

[twi18192@pc0074 docs (tomo-entry-discovery)]$ h5dump -a /entry1/tomo_entry/definition /dls/i12/data/2025/mg37177-1/rawdata/157544.nxs
HDF5 "/dls/i12/data/2025/mg37177-1/rawdata/157544.nxs" {
ATTRIBUTE "definition" {h5dump error: unable to open attribute "definition"

}
}
[twi18192@pc0074 docs (tomo-entry-discovery)]$ h5dump -d /entry1/tomo_entry/definition /dls/i12/data/2025/mg37177-1/rawdata/157544.nxs
HDF5 "/dls/i12/data/2025/mg37177-1/rawdata/157544.nxs" {
DATASET "/entry1/tomo_entry/definition" {
   DATATYPE  H5T_STRING {
      STRSIZE H5T_VARIABLE;
      STRPAD H5T_STR_NULLTERM;
      CSET H5T_CSET_UTF8;
      CTYPE H5T_C_S1;
   }
   DATASPACE  SCALAR
   DATA {
   (0): "NXtomo"
   }
}
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant