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

Adds NXexperiment and NXmeasurement #71

Draft
wants to merge 2 commits into
base: fairmat
Choose a base branch
from

Conversation

domna
Copy link

@domna domna commented Sep 27, 2023

@lukaspie @mkuehbach @RubelMozumder @sherjeelshabih @sanbrock
This is a general experiment and measurement, which I drafted some time ago. Maybe this comes in handy when we build the experiment recipe for #52 . NXmeasurement, is, however, just a collection of thoughts and no valid (n)yaml file, yet.

@lukaspie
Copy link
Collaborator

How would NXexperiment be used? Would our appdefs, e.g. NXmpes, extend NXexperiment? I guess the experiment recipe (at least what I was imagining) is given by NXmeasurement and not really by NXexperiment.

@domna
Copy link
Author

domna commented Sep 27, 2023

How would NXexperiment be used? Would our appdefs, e.g. NXmpes, extend NXexperiment? I guess the experiment recipe (at least what I was imagining) is given by NXmeasurement and not really by NXexperiment.

Yes, this was the idea to have a general NXexperiment to be inherited from in all appdefs to have a general agreed structure.
You are right the recipe is only given by NXmeasurement. I just pushed this here because I had both and used NXmeasurement in NXexperiment, so I thought it's most convenient to have them here together. But we can also separate the NXmeasurement (but this needs some work and polish anyways - just a rough draft).

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.

2 participants