Skip to content

emphasize NXcollection is not validated#1496

Merged
prjemian merged 1 commit intomainfrom
1387-NXcollection-is-not-validated
Oct 15, 2024
Merged

emphasize NXcollection is not validated#1496
prjemian merged 1 commit intomainfrom
1387-NXcollection-is-not-validated

Conversation

@prjemian
Copy link
Copy Markdown
Contributor

@prjemian prjemian requested a review from a team October 15, 2024 16:16
@prjemian prjemian self-assigned this Oct 15, 2024
@prjemian
Copy link
Copy Markdown
Contributor Author

now looks like

image

@prjemian
Copy link
Copy Markdown
Contributor Author

@jkotan - Is this what you had in mind?

@prjemian prjemian added this to the NXDL 2025 milestone Oct 15, 2024
Copy link
Copy Markdown
Contributor

@PeterC-DLS PeterC-DLS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jkotan
Copy link
Copy Markdown
Contributor

jkotan commented Oct 15, 2024

Yes. Does NXpatameters class can be placed in any location below NXentry (similarly to NXcollection)?

@prjemian
Copy link
Copy Markdown
Contributor Author

Yes. It's in the NXentry group docs:
image

@prjemian prjemian merged commit 1dcc838 into main Oct 15, 2024
@prjemian prjemian deleted the 1387-NXcollection-is-not-validated branch October 15, 2024 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DOC: emphasize that content in NXcollection cannot be validated

3 participants