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

Update NeXus search app in pynxtools #580

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

GinzburgLev
Copy link
Contributor

@GinzburgLev GinzburgLev commented Mar 7, 2025

A generic app for search/statistics visualization for NXsensor_scan and inheriting classes (currently NXiv_temp, NXspm, NXstm, NXafm, NXsts); also will work for NOMAD CAMELS entries (most likely, version>=1.8.0).
Please let me know if:

  • NXspm/sts/stm/afm etc should be removed from the filter, as they have a dedicated app with more suitable options
  • NOMAD CAMELS should be mentioned more/less/not at all (currently only menu/protocols(NOMAD CAMELS and protocol column, which is not shown by default) have filters that work only for them)
  • Any functionality / descriptions shall be added or improved
  • Any way to replace explicit list of classes inheriting from NXsensor_scan with something more automated exists

For tests, files from corresponding example uploads will work. Test CAMELS files can be found here: https://box.hu-berlin.de/d/2812ff0b8104478ba160/ (just put any of them in an upload without any extra files/schemas)

@GinzburgLev GinzburgLev requested a review from sanbrock March 7, 2025 16:48
@RubelMozumder
Copy link
Collaborator

You can extend the nexus_app instead of having another app in pynxtools. I think that is what you wanted to do.

@GinzburgLev GinzburgLev changed the title Add new app for NXsensor_scan and inheriting classes (this will also work for NOMAD CAMELS output files) Update NeXus search app in pynxtools Mar 11, 2025
@GinzburgLev
Copy link
Contributor Author

GinzburgLev commented Mar 11, 2025

Now this is not a new app, but an update for existing NeXus app (PR name changed accordingly). I tried to include the comments from today discussion. Several remaining issues that I was not able to fix:

  1. For every widget/menu item except entry_type, if I select more than one option, the search query is A OR B. For entry_type, it is A AND B. I do not know what this depends on.
  2. end time has not been added - currently for start time I use data.ENTRY[*].start_time and not data.ENTRY[*].start_time__field due to definition for NXsensor_scan not yet updated for pynxtools (still wrong format for the field). There is no corresponding field for end_time. Would be easy to do after the updated definition is made available in pynxtools.
  3. Sorting along most columns (such as Start Time) is not possible - according to Lauri, sorting can only be done along scalar fields, so anything within data.ENTRY can not be used for sorting.
  4. Appearance of the dashboard on screens with different resolution: currently looks OK on a large screen (3840x2160, scale=150%) - one line of widgets, multiple entries are visible in the search results. On a reasonably small screen (1920x1080, scale=125%) just having a periodic table and no other widgets is too much - one has to scroll down to see search results. I am not sure what kind of compromise I should take.

Screenshot from 2025-03-11 16-08-42
Screenshot from 2025-03-11 16-08-55
Screenshot from 2025-03-11 16-09-25

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

Successfully merging this pull request may close these issues.

3 participants