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

NXxpcs - proposed application definition #943

Closed
prjemian opened this issue Sep 15, 2021 · 3 comments · Fixed by #982
Closed

NXxpcs - proposed application definition #943

prjemian opened this issue Sep 15, 2021 · 3 comments · Fixed by #982
Assignees
Milestone

Comments

@prjemian
Copy link
Contributor

The US Dept of Energy light sources have been preparing a new application definition for X-ray photon correlation spectroscopy (XPCS) for contribution to NeXus. Progress can be viewed from the group's project page on GitHub.

The NXDL file is expected to reach contribution status this month.

Related to #82

The definition is requesting (issue #932) that NX_COUNT be added as a new type of units.

The definition is requesting (issue #921) that NX_ARBITRARY_UNITS be added as a new type of units.

@prjemian prjemian self-assigned this Sep 15, 2021
@prjemian prjemian added this to the NXDL 2021.10 milestone Sep 15, 2021
@prjemian
Copy link
Contributor Author

The draft NXDL specification is available now: https://github.com/lightsources/BES-XPCS-Pilot/blob/master/NeXus/NXxpcs.nxdl.xml

There is work in branches yet to merged at this time, so expect this is not the final draft.

@benajamin benajamin added telco and removed telco labels Nov 15, 2021
This was referenced Feb 5, 2022
prjemian added a commit to lightsources/definitions that referenced this issue Feb 8, 2022
@prjemian prjemian added the telco label Feb 21, 2022
@prjemian
Copy link
Contributor Author

NXcanSAS avoids a new unit type by use of a named arbitrary units value.

@prjemian
Copy link
Contributor Author

NeXus technical committee advises to use units="unknown" instead of creating a new units type.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants