Skip to content

align with main, groupings in manual - #411

Merged
lukaspie merged 15 commits into
fairmatfrom
fairmat-groupings-in-manual
Nov 10, 2025
Merged

align with main, groupings in manual#411
lukaspie merged 15 commits into
fairmatfrom
fairmat-groupings-in-manual

Conversation

@lukaspie

@lukaspie lukaspie commented Aug 27, 2025

Copy link
Copy Markdown
Collaborator

This brings back various changes from the nexusformat main branch, including the groupings in the manual from nexusformat#1428

@lukaspie
lukaspie changed the base branch from main to fairmat August 27, 2025 16:20
@lukaspie
lukaspie force-pushed the fairmat-groupings-in-manual branch from 348aab1 to e6f7bca Compare August 29, 2025 10:18
@lukaspie
lukaspie force-pushed the fairmat-groupings-in-manual branch from e6f7bca to 817f05f Compare October 23, 2025 15:29
@lukaspie
lukaspie force-pushed the fairmat-groupings-in-manual branch from 30e4c1d to 8598de0 Compare October 24, 2025 07:50
@lukaspie lukaspie changed the title [DO NOT MERGE] groupings in manual align with main, groupings in manual Oct 24, 2025
@lukaspie
lukaspie requested a review from mkuehbach October 24, 2025 08:17
Comment thread .cspell/custom-dictionary.txt
Comment thread applications/NXapm.nxdl.xml
Comment thread applications/NXtomoproc.nxdl.xml
Comment thread base_classes/NXentry.nxdl.xml

@mkuehbach mkuehbach left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I support the PR for what it is, bringing the state of the official NIAC main back to our fairmat branch so that the diffs between the NXDL2025 release and our fairmat branch are minimal.

There are a few minor glitches where also the NIAC main branch would profit from being fixed but agree that this should also not block the release, specifically not when it mostly touches on contributed definitions.

Comment thread contributed_definitions/NXdispersive_material.nxdl.xml
Comment thread contributed_definitions/NXdispersion_table.nxdl.xml
Comment thread contributed_definitions/NXdispersion.nxdl.xml
Comment thread contributed_definitions/NXoptical_fiber.nxdl.xml
Comment thread contributed_definitions/NXoptical_polarizer.nxdl.xml
Comment thread contributed_definitions/NXoptical_polarizer.nxdl.xml Outdated

@mkuehbach mkuehbach left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

One inconsistence to fix and then this can be merged.

Comment thread base_classes/NXelectron_detector.nxdl.xml
@lukaspie
lukaspie merged commit 732f57a into fairmat Nov 10, 2025
6 checks passed
@lukaspie
lukaspie deleted the fairmat-groupings-in-manual branch November 10, 2025 16:03
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 this pull request may close these issues.

2 participants