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.

Sign up for GitHub

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

Jump to bottom

NXmx has fields that may not have to be required #696

Closed
PeterC-DLS opened this issue Jan 21, 2020 · 0 comments
Closed

NXmx has fields that may not have to be required #696

PeterC-DLS opened this issue Jan 21, 2020 · 0 comments
Assignees
@PeterC-DLS
Milestone
NXDL 2020.1

Comments

@PeterC-DLS
Copy link
Contributor

NXmx has a NXsource with many fields that are required. Perhaps these have been mistakenly marked as such and should be optional as they do not apply for synchrotron sources.
@PeterC-DLS PeterC-DLS added this to the NXDL 2019.10 milestone Jan 21, 2020
@PeterC-DLS PeterC-DLS self-assigned this Jan 22, 2020
PeterC-DLS added a commit that referenced this issue Jan 22, 2020
@PeterC-DLS
Set minOccurs to zero for unneeded items in NXsource of NXmx
3ef1bbd 
Fix #696 as documentation for application definitions adds items
as required if they do not set minOccurs as zero
@PeterC-DLS PeterC-DLS removed their assignment Jan 22, 2020
@PeterC-DLS PeterC-DLS mentioned this issue Jan 22, 2020
Closed
@PeterC-DLS PeterC-DLS self-assigned this Jan 22, 2020
@PeterC-DLS PeterC-DLS mentioned this issue Jan 23, 2020
Merged
@prjemian prjemian mentioned this issue Jan 26, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PeterC-DLS PeterC-DLS

Labels
None yet
Projects
None yet
Milestone
NXDL 2020.1
Development

Successfully merging a pull request may close this issue.

Set minOccurs to zero for unneeded items in NXsource of NXmx nexusformat/definitions
1 participant
@PeterC-DLS