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

Show NXentry title in tree #1622

Open
woutdenolf opened this issue Apr 23, 2024 · 3 comments
Open

Show NXentry title in tree #1622

woutdenolf opened this issue Apr 23, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@woutdenolf
Copy link

NXentry names are often adapted for machine consumption while the title is meant for human consumption.

Silx view shows both

image

H5web does not

image

It does mean you would need to fetch the "title" dataset if any which you might not want to do? Or perhaps make it optional?

@woutdenolf woutdenolf added the enhancement New feature or request label Apr 23, 2024
@loichuder
Copy link
Member

Hm. How far would you like to go with this ?

Because if I look at our beloved water_224.h5 file in silx view, the description column can be filled in many ways:
image

  • NXentry entry_0000: title dataset
  • NXprocess 1_integration: program dataset
  • NXinstrument BM20: name dataset
  • NXsample water: description dataset

@axelboc
Copy link
Contributor

axelboc commented Apr 24, 2024

It does mean you would need to fetch the "title" dataset if any which you might not want to do? Or perhaps make it optional?

Indeed, for each "NX" badge, we have access only to the metadata of the group itself (i.e. dataset01, dataset02, etc.), including attributes, but not the metadata of its children. We could fetch the metadata of the children as well, but this may lead to requests pile-ups in files/groups with lots of NX sub-groups, thus slowing down fetching actual data. It gets worse when considering nested NX entities, as Loïc mentioned.

I could also imagine cases where the groups would be named meaningfuly and the titles would be generic...

Maybe there's a compromise to be found in naming the datasets, like 01_APS-13-ID-C_Co_K? In a way, it's not really helping machines either (including H5Web) if, to get to a meaningful name, they have to know about NeXus and dig through the tree to find a distinctive piece of information. 😝

@woutdenolf
Copy link
Author

How far would you like to go with this ?

Just the NXentry title.

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

No branches or pull requests

3 participants