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

Refactor HEDM calibration to use new workflow #1800

Merged
merged 1 commit into from
Mar 7, 2025

Conversation

psavery
Copy link
Collaborator

@psavery psavery commented Mar 7, 2025

This refactors the HEDM calibration to use the new calibration workflow we have been using for powder and Laue. It includes the use of lmfit for boundary constraints, support for relative constraints between detectors (including for subpanels), and many highly desirable features such as undo/redo.

This refactoring completely eliminates the need for the old calibration workflow, and thus a lot of old code is being removed as well, including the ability to set refinements in the instrument tree view and the refinements editor.

Although this still needs to be further tested, it is overall a major improvement for HEDM calibration.

Depends on: HEXRD/hexrd#779

This refactors the HEDM calibration to use the new calibration workflow
we have been using for powder and Laue. It includes the use of lmfit
for boundary constraints, support for relative constraints between
detectors (including for subpanels), and many highly desirable
features such as undo/redo.

This refactoring completely eliminates the need for the old calibration
workflow, and thus a lot of old code is being removed as well, including
the ability to set refinements in the instrument tree view and the
refinements editor.

Although this still needs to be further tested, it is overall a major
improvement for HEDM calibration.

Depends on: HEXRD/hexrd#779

Signed-off-by: Patrick Avery <[email protected]>
@psavery
Copy link
Collaborator Author

psavery commented Mar 7, 2025

Since this is somewhat urgent, we'll get it in and perform any necessary cleanup later...

@psavery psavery merged commit b7109c5 into master Mar 7, 2025
9 of 15 checks passed
@psavery psavery deleted the refactor-hedm-calibration branch March 7, 2025 14:33
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.

1 participant