merge: pydantic refactor & CHAP.saxswaxs#231
Merged
keara-soloway merged 102 commits intosaxswaxsfrom Feb 11, 2026
Merged
Conversation
With so far only illustrating the use of common.ImageProcessor
This reverts commit a1bd192.
This reverts commit 47b4d75.
- Added CHAP.pipeline.PipelineItem.validate(), which validates the pipeline, similar to how execute() executes it. - Created CHAP.common.reader.ConfigReader, which reads and verifies a config file in yaml formet - Make inputdir, outputdit, interactive and schema attributes of a PipelineItem object. This mostly avoids needing to specify them paramaters to reader, processors, or writers.
Also updated the test cases to run tests with the latest Tomo code
Finished most of the readers/writers Todo: CHAP.common.reader.NXdataReader CHAP.common.reader.NXfieldReader CHAP.common.reader.URLReader The FOXDEN readers with config parameters The FOXDEN writers with url parameter
docs: fixed "make html" error in Sphinx autodoc
build: update EDD environment to prep testing EDD after code updates
update to newest packages for python 3.10
doc: made fixes to issues found during deployment
Only defined in NXentry or NXroot in the current Nexusformat definition
Done to mimic the behavior of rel_height_cutoff, but only for the scipy find_peaks part that pick the peaks to fit for each detector based on their height relative to the average for that detector
The default NXdata object for each detector now also has the stdevs in the unconstrained strains obtained from the peak positions relative to their nominal values from the Bragg relation for the HKLs included in the fits
Showed only up for a single data point in the map
fix: bug in EDD strain mean data calculation
Fixed MapProcessor and EDD when all independent dims in the map are constant
Also update vrange in common.models.map.ImageProcessorConfig to allow users to set only the lower or upper bound (cherry picked from commit c9b4d74)
As needed to get this info, a best_vary property was added to CHAP.utils.fit.Fit and CHAP.utils.fit.FitMap which returns the `vary` parameter for the best fit parameters.
The default NXdata object for each detector now also has the stdevs in the unconstrained strains obtained from the peak positions relative to their nominal values from the Bragg relation for the HKLs included in the fits (cherry picked from commit b0013ae)
(cherry picked from commit 19f3b45)
For now set the fit to be performed using lmfit
fix: rel_height_cutoff>0 in EDD strain analysis not working with scipy
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.