Ticket #6230 (closed: duplicate)

Opened 8 years ago

Last modified 5 years ago

Modify absolute normalization to produce correct cross-section

Reported by: Alex Buts Owned by: Alex Buts
Priority: major Milestone: Release 2.6
Component: Framework Keywords:
Cc: Blocked By:
Blocking: #5567 Tester: Michael Reuter

Description

The accuracy of monovanadium integral depends on sufficient statistics collected in the monovanadium scattering experiment.

Insufficient statistics causes distribution to be non-Gaussian and Bayesian statistics needs to be deployed to estimate correct averages.

The purpose of the ticket is:

When absolute normalization is used:

1) Estimate the statistics used for monovan integration and warn the user if this statistics is insufficient.

2) use best Bayesian approach in this situation

3) Write the tests to verify the vanadium cross section is calculated properly.

Change History

comment:1 Changed 8 years ago by Alex Buts

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Alex Buts

  • Blocking 5567 added

comment:3 Changed 8 years ago by Alex Buts

refs #6230 Modified SumSpectra algorithm

to return statistics on the sum

Changeset: 5b2fc2d835beff2b7a6ce831334f2e2054cb41ca

comment:4 Changed 8 years ago by Alex Buts

refs #6230 Method to get min-max values on axis

to estimate integration range etc and exported these functions to python

Changeset: 996924cc88f0627f85bb4e51993b1180e5413ea4

comment:5 Changed 8 years ago by Alex Buts

refs #6230 Modified sum spectra to produce weighted sum on request

Changeset: b04a8108ea15e21f97d70aa98875647fb63d5890

comment:6 Changed 8 years ago by Alex Buts

refs #6230 Modified sum spectra

to be capable of producing weighted sum

Changeset: 1986488bac1f7bde75d2bf98c06a1ae673cf5183

comment:7 Changed 8 years ago by Alex Buts

refs #6230 working Weighted averages with more tests

Changeset: edc2659c36843849679ab6dc124adb8f9d68540d

comment:8 Changed 8 years ago by Alex Buts

refs #6230 Removed deprecated Libisis summation

Changeset: 99d0399a51a3dfbcc8188c8b8fe5f114020643e9

comment:9 Changed 8 years ago by Alex Buts

refs #6230 fixing Unix warnings

Changeset: a65ed48fd10ec5498f7db8990cb631c42c5f8724

comment:10 Changed 8 years ago by Alex Buts

refs #6230 small comments

Changeset: b6f7230906a9af60b3aca23292cdff0c6a1369b1

comment:11 Changed 8 years ago by Alex Buts

refs #6230 Modified SumSpectra algorithm

to return statistics on the sum

Changeset: 5b2fc2d835beff2b7a6ce831334f2e2054cb41ca

comment:12 Changed 8 years ago by Alex Buts

refs #6230 Method to get min-max values on axis

to estimate integration range etc and exported these functions to python

Changeset: 996924cc88f0627f85bb4e51993b1180e5413ea4

comment:13 Changed 8 years ago by Alex Buts

refs #6230 Modified sum spectra to produce weighted sum on request

Changeset: b04a8108ea15e21f97d70aa98875647fb63d5890

comment:14 Changed 8 years ago by Alex Buts

refs #6230 Modified sum spectra

to be capable of producing weighted sum

Changeset: 1986488bac1f7bde75d2bf98c06a1ae673cf5183

comment:15 Changed 8 years ago by Alex Buts

refs #6230 working Weighted averages with more tests

Changeset: edc2659c36843849679ab6dc124adb8f9d68540d

comment:16 Changed 8 years ago by Alex Buts

refs #6230 fixing Unix warnings

Changeset: a65ed48fd10ec5498f7db8990cb631c42c5f8724

comment:17 Changed 8 years ago by Alex Buts

refs #6230 small comments

Changeset: b6f7230906a9af60b3aca23292cdff0c6a1369b1

comment:18 Changed 8 years ago by Alex Buts

  • Milestone changed from Release 2.4 to Release 2.5

comment:19 Changed 7 years ago by Alex Buts

  • Milestone changed from Release 2.5 to Release 2.6

comment:20 Changed 7 years ago by Alex Buts

  • Status changed from accepted to verify
  • Resolution set to duplicate

there was some job done on this ticket, but it all was incorporated and completed within the changes done within ticket #6667.

I have reduced vanadium runs on all four ISIS inelastic instruments and the cross-sections they produce are correct. There are possibilities for improvements in the code and in the work-flow, but they will be addressed by separate tickets.

The results of the reduction were discussed with instrument scientists and are accepted. There is nothing to test here.

comment:21 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:22 Changed 7 years ago by Michael Reuter

  • Status changed from verifying to closed

Yes, it looks like the changes have been in use for quite some time now.

comment:23 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:24 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7076

Note: See TracTickets for help on using tickets.