Closed RichardWaiteSTFC closed 5 months ago
4 files 106 suites 13m 30s :stopwatch: 684 tests 653 :white_check_mark: 18 :zzz: 1 :x: 12 :fire: 1 906 runs 1 854 :white_check_mark: 36 :zzz: 4 :x: 12 :fire:
For more details on these failures and errors, see this check.
Results for commit 370fb8cc.
:recycle: This comment has been updated with latest results.
This issue was found during testing of #164 - when fitting powder spectra with resolution convolution, performing the convolution in energy after the binning was leading to large discontinuities in the cost function derivatives wert the parameters.
This PR accounts for the energy resolution 'properly' by considering the resolution at the point the intensity at each eigenvalue is binned onto a grid. Note it assumes the energy resolution is Gaussian and is separable from the Q resolution.