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

Add nexus definitions for hextof in the repo #417

Closed
zain-sohail opened this issue Jun 14, 2024 · 4 comments · Fixed by #420
Closed

Add nexus definitions for hextof in the repo #417

zain-sohail opened this issue Jun 14, 2024 · 4 comments · Fixed by #420

Comments

@zain-sohail
Copy link
Member

NXmpes_config-HEXTOF_full_Feb2024-1.json

Here's the definiton provided by @kutnyakhov . Might need updating.

@zain-sohail
Copy link
Member Author

Should also save the processing steps in the file because currently they are not available when exported to nexus.

@rettigl
Copy link
Member

rettigl commented Jun 14, 2024

Should also save the processing steps in the file because currently they are not available when exported to nexus.

Yes, the processing metadata are mostly also still missing for mpes. Would be good to create also an example for the documentation for this, if possible (not sure how to handle the SciCat part there, however).

@zain-sohail
Copy link
Member Author

Yes, the processing metadata are mostly also still missing for mpes. Would be good to create also an example for the documentation for this, if possible (not sure how to handle the SciCat part there, however).

So you mean we create a documentation of how to make your nexus definition json? I am not fully sure how since Dima did it but I can take a look.

Regarding Scicat, DESY users should be able to access scicat with their account and get a token.
For documentation, we can provide the json file that scicat returns.

@rettigl
Copy link
Member

rettigl commented Jun 18, 2024

I opend a PR for discussion of this: #420

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 a pull request may close this issue.

2 participants