Ticket #4923 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Errors of MuonAsymmetry algorithms cannot be fully correct

Reported by: Anders Markvardsen Owned by: Anders Markvardsen
Priority: critical Milestone: Release 2.1
Component: Mantid Keywords:
Cc: robert.whitley@… Blocked By:
Blocking: Tester: Alex Buts

Description

Email has been sent to Steve since a formula he provided does not work when data are bunched.

The email is asking him to double check his error calculation formula.

Change History

comment:1 Changed 9 years ago by Anders Markvardsen

Fixed a subtle bug when data are bunched in MuonAnalysis. re #4923

Changeset: 0ccfad6c6717889888fba4ce8445615cd808069e

comment:2 Changed 9 years ago by Anders Markvardsen

Remove last bunched bin if shorter than previous bins. re #4923

Changeset: 0774fffb49cbc64fc4c58e412b7ad65fb596ae3a

comment:3 Changed 9 years ago by Anders Markvardsen

Fixed a subtle bug when data are bunched in MuonAnalysis. re #4923

Changeset: 0ccfad6c6717889888fba4ce8445615cd808069e

comment:4 Changed 9 years ago by Anders Markvardsen

Remove last bunched bin if shorter than previous bins. re #4923

Changeset: 0774fffb49cbc64fc4c58e412b7ad65fb596ae3a

comment:5 Changed 9 years ago by Russell Taylor

Clear compiler warning. Re #4923.

Changeset: 92a262905804f2ba0ff9cf5d2bf6b71726a95c61

comment:6 Changed 9 years ago by Anders Markvardsen

  • Status changed from new to accepted

The formula turned out to be correct but it is calculating the errors as square root of the counts and in that way ignoring the actual errors in the data. Steve prefers it that way.

When bunching we did this by averaging rather than summing the counts.

Bunching with this ticket has now been changed to sum counts, which at the same time means that the current error cal in MuonAsymmetry algorithms is now valid.

To check simply plot pair asymmetry data in MuonAnalysis with increasing bunching at watch the errorbars to get smaller

comment:7 Changed 9 years ago by Anders Markvardsen

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

comment:8 Changed 8 years ago by Alex Buts

  • Status changed from verify to verifying
  • Tester set to Alex Buts

comment:9 Changed 8 years ago by Alex Buts

Errors indeed become smaller. works and looks correct visually

leave exact calculations to a scientist

comment:10 Changed 8 years ago by Alex Buts

  • Status changed from verifying to closed

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5769

Note: See TracTickets for help on using tickets.