MPES: new concepts from NIAC discussions, searchable fields #329
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Explicitly name the following axes in
NXmpes/NXentry/data(NXdata)
:Explicitly name the following axes in
NXmpes/NXinstrument/NXelectronanalyser/NXdetector/raw_data(NXdata)
:NXprocess and subclasses are stored directly under
Nxmpes/NXentry
AXIS_calibration
for general calibration on all axes that are defined inNXdata
NXprocess_mpes
As a consequence, remove base classes
NXdata_mpes
,NXdata_mpes_detector
, andNXprocess_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 docsMention the following terms as optional in the application definition:
change all instances of
NXidentifier
to fields inheriting frominstanceNAME
remove
NXsubstance
, use existing concepts inNXsample
for nowNXsample/chemical_formula
add field
core_level
under ENTRYlist like
NXdata/@axes
suggested enumeration of all possible core levels
Open questions:
k_parallel
,k_perpendicular
?