This work comprises 2 improvements to single-crystal integration methods:
When determining background bins by minimising third moment (skew) in IntegratePeaksSkew - ensure loop terminates if skew < 0 (we expect minimum skew of background to be 0 - for low counts it should in fact be positive)
Add option to ignore peaks for which mask (or shoebox etc.) includes a masked detector - this affects
IntegratePeaksSkew
IntegratePeaksShoeboxTOF
IntegratePeaks1DProfile
Purpose of work
Change (1) stops peak mask of IntegratePEaksSkew including too many pixels due to small diffuse scattering - see issue #37105
Change (2) allows the user to make a custom mask for edges (or dodgy tube etc.) - whereas NcolsEdge and NRowsEdge are assumed to be the same for all banks. To support this functionality the minimum value for NrowsEdge and NColsEdge has been decreased to 0 - i.e. it is now possible to only define edges using masked detectors on the workspace. Note this is how detector edges are denoted in other integration algorithms such asIntegratePeaksMD.
(2) Open the pdf output and check the mask still covers the peaks (should look like the left hand column in the screenshots above)
Reviewer
Please comment on the points listed below (full description).
Your comments will be used as part of the gatekeeper process, so please comment clearly on what you have checked during your review. If changes are made to the PR during the review process then your final comment will be the most important for gatekeepers. In this comment you should make it clear why any earlier review is still valid, or confirm that all requested changes have been addressed.
Do changes function as described? Add comments below that describe the tests performed?
Do the changes handle unexpected situations, e.g. bad input?
Has the relevant (user and developer) documentation been added/updated?
Does everything look good? Mark the review as Approve. A member of @mantidproject/gatekeepers will take care of it.
Gatekeeper
If you need to request changes to a PR then please add a comment and set the review status to "Request changes". This will stop the PR from showing up in the list for other gatekeepers.
Description of work
This work comprises 2 improvements to single-crystal integration methods:
IntegratePeaksSkew
- ensure loop terminates if skew < 0 (we expect minimum skew of background to be 0 - for low counts it should in fact be positive)IntegratePeaksSkew
IntegratePeaksShoeboxTOF
IntegratePeaks1DProfile
Purpose of work
Change (1) stops peak mask of
IntegratePEaksSkew
including too many pixels due to small diffuse scattering - see issue #37105Change (2) allows the user to make a custom mask for edges (or dodgy tube etc.) - whereas
NcolsEdge
andNRowsEdge
are assumed to be the same for all banks. To support this functionality the minimum value forNrowsEdge
andNColsEdge
has been decreased to 0 - i.e. it is now possible to only define edges using masked detectors on the workspace. Note this is how detector edges are denoted in other integration algorithms such asIntegratePeaksMD
.Fixes #37105
To test:
(1) Run this script
(2) Open the pdf output and check the mask still covers the peaks (should look like the left hand column in the screenshots above)
Reviewer
Please comment on the points listed below (full description). Your comments will be used as part of the gatekeeper process, so please comment clearly on what you have checked during your review. If changes are made to the PR during the review process then your final comment will be the most important for gatekeepers. In this comment you should make it clear why any earlier review is still valid, or confirm that all requested changes have been addressed.
Code Review
Functional Tests
Does everything look good? Mark the review as Approve. A member of
@mantidproject/gatekeepers
will take care of it.Gatekeeper
If you need to request changes to a PR then please add a comment and set the review status to "Request changes". This will stop the PR from showing up in the list for other gatekeepers.