MPES: new concepts from NIAC discussions, searchable fields

[lukaspie]

• Explicitly name the following axes in NXmpes/NXentry/data(NXdata):

  • energy
  • photon energy
  • kx, ky, kz
  • k_parallel, k_perpendicular
  • angular0, angular1
  • spatial0,spatial1
  • delay
  • temperature

• Explicitly name the following axes in NXmpes/NXinstrument/NXelectronanalyser/NXdetector/raw_data(NXdata):

  • pixel_x, pixel_y
  • photon energy
  • kx, ky, kz
  • k_parallel, k_perpendicular
  • angular0, angular1
  • spatial0,spatial1
  • delay
  • temperature

• NXprocess and subclasses are stored directly under Nxmpes/NXentry

  • AXIS_axis_calibration for general calibration on all axes that are defined in NXdata
  • Brought in special calibrations from existing base class NXprocess_mpes

• As a consequence, remove base classes NXdata_mpes , NXdata_mpes_detector, and NXprocess_mpes

• Allow for multiple beams and sources

  • beam_probe/source_probe
  • beam_pump/source_probe
  • beamTYPE/sourceTYPE -> I still think this is valuable, see the description in its docs

• change all instances of NXidentifier to fields inheriting from identifierNAME

• remove NXsubstance, use existing concepts in NXsample for now

  • use NXsample/chemical_formula

• add field core_levels under ENTRY

  • list like NXdata/@axes
  • suggested enumeration of all possible core levels

• make all NXtransformations/@vector attributes NX_NUMBER

• add explicit open enumeration for function_type for the peak and background in NXxps/NXfit

Open questions:

• Do we need k_parallel, k_perpendicular?
• Is AXIS_axis_calibration good enough or do we need individual named calibrations for each axis?
• Can we improve core_level description?

[rettigl]

_indices are INT

[rettigl]

There are also still NXidentifiers in the base classes, e.g. NXcalibration

[rettigl]

I fixed some of these spellings in the yaml files, the xmls need to be generated.

[lukaspie]

There are also still NXidentifiers in the base classes, e.g. NXcalibration

This will come with #332 (merging back from NIAC main and removing classes that were not accepted). I will rebase this branch here afterwards.

[lukaspie]

@rettigl do we still need this comment?

[rettigl]

@rettigl do we still need this comment?

Don't think so.

[lukaspie]

I remember you saying NXcoordinate_system_set should be renamed or replaced?

Correct, we will just not use it anymore (similar to having no grouping for Nxprocess). The coordinate systems are now directly under NXentry.

[lukaspie]

Is there a convention regarding the use of en-US vs. en-GB in Nexus? Our definitions are somewhat inconsistent here, e.g. "NXelectronanalyser" vs. "analyzer" at several places in the docstrings. I would suggest fixing this to one convention. Another example is "oxidising" vs. oxidizing.

There is not really an official guideline. Most of the documentation uses American English, but there are some other cases as well. Maybe @sanbrock has an idea?

[lukaspie]

Is there a convention regarding the use of en-US vs. en-GB in Nexus? Our definitions are somewhat inconsistent here, e.g. "NXelectronanalyser" vs. "analyzer" at several places in the docstrings. I would suggest fixing this to one convention. Another example is "oxidising" vs. oxidizing.

I went through the existing definitions in the NIAC repo, it seems like in almost all cases, the en-US version is used (e.g. optimization, minimization, etc.). It also makse sense since most NIAC members have been from the US and only a few of them from the UK.

If you agree, I would change all of our definiitions to be en-US, as well.

[rettigl]

If you agree, I would change all of our definiitions to be en-US, as well.

Yes, please.

[lukaspie]

If you agree, I would change all of our definiitions to be en-US, as well.

Yes, please.

Done, see the last two commits. Main difference was analyser -> analyzer. Our tools will have to adjust to that as well.

As an addition, I also removed language like "NXenergydispersion is a sublass of NXelectronanalyzer", since it is not actually a subclass (in the inheritance sense), it is just used within.

If you don't have anything else to change @rettigl, I would merge here and bring the NIAC PR uptodate as well.

[lukaspie]

@rettigl this was apparently lost at some point, but I readded it here