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

Move NXpid to base classes #1522

Draft
wants to merge 25 commits into
base: main
Choose a base branch
from
Draft

Conversation

lukaspie
Copy link
Contributor

This is a follow-up to #1414 where NXpid was mistakenly not included in the vote even though it was originally used within NXactuator. This PR moves NXpid to the base classes and adds it back to NXactuator. It will be rebased as soon as #1414 is reviewed and merged.

lukaspie and others added 25 commits December 11, 2024 13:39
# Conflicts:
#	base_classes/NXenvironment.nxdl.xml
#	base_classes/NXinstrument.nxdl.xml
#	base_classes/nyaml/NXinstrument.yaml
#	contributed_definitions/NXmanipulator.nxdl.xml
#	contributed_definitions/NXmpes.nxdl.xml
#	contributed_definitions/nyaml/NXmanipulator.yaml
#	contributed_definitions/nyaml/NXmpes.yaml
# Conflicts:
#	contributed_definitions/NXactuator.nxdl.xml
#	contributed_definitions/NXelectron_level.nxdl.xml
#	contributed_definitions/NXmpes.nxdl.xml
#	contributed_definitions/nyaml/NXactuator.yaml
# Conflicts:
#	base_classes/nyaml/NXenvironment.yaml
# Conflicts:
#	base_classes/nyaml/NXenvironment.yaml
#	contributed_definitions/NXactuator.nxdl.xml
#	contributed_definitions/NXcalibration.nxdl.xml
#	contributed_definitions/NXenergydispersion.nxdl.xml
#	contributed_definitions/NXmanipulator.nxdl.xml
#	contributed_definitions/NXpid.nxdl.xml
#	contributed_definitions/NXresolution.nxdl.xml
#	contributed_definitions/nyaml/NXactuator.yaml
#	contributed_definitions/nyaml/NXcalibration.yaml
#	contributed_definitions/nyaml/NXcollectioncolumn.yaml
#	contributed_definitions/nyaml/NXelectronanalyser.yaml
#	contributed_definitions/nyaml/NXenergydispersion.yaml
#	contributed_definitions/nyaml/NXmanipulator.yaml
#	contributed_definitions/nyaml/NXpid.yaml
#	contributed_definitions/nyaml/NXresolution.yaml
# Conflicts:
#	base_classes/nyaml/NXenvironment.yaml
#	contributed_definitions/NXmpes.nxdl.xml
#	contributed_definitions/nyaml/NXmpes.yaml
…ersion

# Conflicts:
#	applications/NXarpes.nxdl.xml
#	applications/nyaml/NXarpes.yaml
#	base_classes/NXaperture.nxdl.xml
#	base_classes/NXbeam.nxdl.xml
#	base_classes/NXdata.nxdl.xml
#	base_classes/NXdetector.nxdl.xml
#	base_classes/NXentry.nxdl.xml
#	base_classes/NXinstrument.nxdl.xml
#	base_classes/NXmonochromator.nxdl.xml
#	base_classes/NXroot.nxdl.xml
#	base_classes/NXsample.nxdl.xml
#	base_classes/NXsample_component.nxdl.xml
#	base_classes/NXsensor.nxdl.xml
#	base_classes/NXsource.nxdl.xml
#	base_classes/NXsubentry.nxdl.xml
#	base_classes/NXtransformations.nxdl.xml
#	base_classes/NXuser.nxdl.xml
#	base_classes/nyaml/NXaperture.yaml
#	base_classes/nyaml/NXbeam.yaml
#	base_classes/nyaml/NXdata.yaml
#	base_classes/nyaml/NXdetector.yaml
#	base_classes/nyaml/NXentry.yaml
#	base_classes/nyaml/NXenvironment.yaml
#	base_classes/nyaml/NXinstrument.yaml
#	base_classes/nyaml/NXmonochromator.yaml
#	base_classes/nyaml/NXprocess.yaml
#	base_classes/nyaml/NXroot.yaml
#	base_classes/nyaml/NXsample.yaml
#	base_classes/nyaml/NXsample_component.yaml
#	base_classes/nyaml/NXsensor.yaml
#	base_classes/nyaml/NXsource.yaml
#	base_classes/nyaml/NXsubentry.yaml
#	base_classes/nyaml/NXtransformations.yaml
#	base_classes/nyaml/NXuser.yaml
# Conflicts:
#	base_classes/NXdetector.nxdl.xml
#	base_classes/NXinstrument.nxdl.xml
#	base_classes/NXsensor.nxdl.xml
#	base_classes/NXsource.nxdl.xml
# Conflicts:
#	base_classes/NXinstrument.nxdl.xml
#	base_classes/nyaml/NXenvironment.yaml
…s in the catchen test

# Conflicts:
#	base_classes/NXdata.nxdl.xml
…ersion

# Conflicts:
#	applications/NXarpes.nxdl.xml
#	applications/nyaml/NXarpes.yaml
#	base_classes/NXaperture.nxdl.xml
#	base_classes/NXbeam.nxdl.xml
#	base_classes/NXdata.nxdl.xml
#	base_classes/NXdetector.nxdl.xml
#	base_classes/NXentry.nxdl.xml
#	base_classes/NXenvironment.nxdl.xml
#	base_classes/NXinstrument.nxdl.xml
#	base_classes/NXmonochromator.nxdl.xml
#	base_classes/NXroot.nxdl.xml
#	base_classes/NXsample.nxdl.xml
#	base_classes/NXsample_component.nxdl.xml
#	base_classes/NXsensor.nxdl.xml
#	base_classes/NXsource.nxdl.xml
#	base_classes/NXsubentry.nxdl.xml
#	base_classes/NXtransformations.nxdl.xml
#	base_classes/NXuser.nxdl.xml
#	base_classes/nyaml/NXaperture.yaml
#	base_classes/nyaml/NXbeam.yaml
#	base_classes/nyaml/NXdata.yaml
#	base_classes/nyaml/NXdetector.yaml
#	base_classes/nyaml/NXentry.yaml
#	base_classes/nyaml/NXenvironment.yaml
#	base_classes/nyaml/NXinstrument.yaml
#	base_classes/nyaml/NXmonochromator.yaml
#	base_classes/nyaml/NXprocess.yaml
#	base_classes/nyaml/NXroot.yaml
#	base_classes/nyaml/NXsample.yaml
#	base_classes/nyaml/NXsample_component.yaml
#	base_classes/nyaml/NXsensor.yaml
#	base_classes/nyaml/NXsource.yaml
#	base_classes/nyaml/NXsubentry.yaml
#	base_classes/nyaml/NXtransformations.yaml
#	base_classes/nyaml/NXuser.yaml
bring back files lost during cherry pick
Co-authored-by: Aaron S. Brewster <[email protected]>
@lukaspie lukaspie changed the title Nxpid Move Nxpid to base classes Dec 11, 2024
@lukaspie
Copy link
Contributor Author

Currently blocked by review on #1414.

@lukaspie lukaspie added NIAC should review The NIAC should review/discuss NIAC vote needed PR needs an approving vote from NIAC before merge labels Dec 11, 2024
@lukaspie lukaspie changed the title Move Nxpid to base classes Move NXpid to base classes Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion needed enhancement NIAC should review The NIAC should review/discuss NIAC vote needed PR needs an approving vote from NIAC before merge
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

4 participants