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

check the sources for instances where nameType (UPPERCASE flexible names) rules are broken #1490

Open
Tracked by #1485
prjemian opened this issue Oct 1, 2024 · 3 comments
Labels
NIAC has requested The NIAC has requested this issue to be considered
Milestone

Comments

@prjemian
Copy link
Contributor

prjemian commented Oct 1, 2024

Handle action item here as separate issue, per NIAC2024 approval of nameType attribute.

List where a nameType attribute is needed. Could be a list of each NXDL file. Then open new issue for each.

@prjemian prjemian added the NIAC has requested The NIAC has requested this issue to be considered label Oct 1, 2024
@prjemian prjemian added this to the NXDL 2025 milestone Oct 1, 2024
@prjemian
Copy link
Contributor Author

prjemian commented Oct 1, 2024

As noted, in NXinstrument

In Nxinstrument I see

Should be <group type="NXtransformations" name="DIFFRACTOMETER" nameType="any" />

@lukaspie
Copy link
Contributor

NXtransformations has fields AXISNAME_end and AXISNAME_increment_set (right below), these should be nameType="partial".

@lukaspie
Copy link
Contributor

Another one: NXentry/definition/@URL should have nameType="specified", similar to the uppercase names in NXcanSAS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NIAC has requested The NIAC has requested this issue to be considered
Projects
Status: No status
Development

No branches or pull requests

2 participants