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

AppDef for Multidimensional photoemission spectroscopy (MPES) #105

Open
sanbrock opened this issue Mar 2, 2022 · 4 comments
Open

AppDef for Multidimensional photoemission spectroscopy (MPES) #105

sanbrock opened this issue Mar 2, 2022 · 4 comments
Assignees

Comments

@sanbrock
Copy link

sanbrock commented Mar 2, 2022

https://github.com/FAIRmat-Experimental/nexus_definitions/tree/fairmat

@prjemian
Copy link
Contributor

prjemian commented Mar 2, 2022

The new definition should be added to the contributed_definitions folder of the definitions repository (in a branch or fork). Once ratified by the NIAC (item 3 of NeXus Constitution), it will be moved to the applications folder.

The pull request should be made to the definitions repository.

@sanbrock
Copy link
Author

sanbrock commented Mar 3, 2022

@benajamin
Copy link
Contributor

I notice that some fields in NXmanipulator would be appropriate to have in NXsample. Is there a reason to choose one location over another? Perhaps having a policy of using links so that they are found in both locations would be useful?

@Tommaso-Pincelli
Copy link

I notice that some fields in NXmanipulator would be appropriate to have in NXsample. Is there a reason to choose one location over another? Perhaps having a policy of using links so that they are found in both locations would be useful?

NXmanipulator is a rough draft. It does not contain the depends_on/NXtransformations structure yet (which we have implemented in our local versions). We created the class because in most photoemission experiments the manipulator is a complex independent experimental subsystem with a lot of functionalities. Clearly it can be linked into NXsample. We are debating if it should:

  1. Disappear by merging with NXstage
  2. Be a sub-class of NXsample
  3. Be indipendent

@sanbrock sanbrock self-assigned this Sep 14, 2022
@sanbrock sanbrock moved this to Done in NIAC2022 project Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

No branches or pull requests

4 participants