New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix period subtraction in MuonCalculateAsymmetry #14018
Merged
KarlPalmen
merged 13 commits into
master
from
13952_Fix_period_subtraction_in_MuonCalculateAsymmetry
Oct 20, 2015
Merged
Fix period subtraction in MuonCalculateAsymmetry #14018
KarlPalmen
merged 13 commits into
master
from
13952_Fix_period_subtraction_in_MuonCalculateAsymmetry
Oct 20, 2015
Conversation
This file contains 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
Now the calculation is right for - operation but wrong for +
…_Fix_period_subtraction_in_MuonCalculateAsymmetry
raquelalvarezbanos
added
the
Patch Candidate
Urgent issues that must be included in a patch following a release
label
Oct 16, 2015
…_Fix_period_subtraction_in_MuonCalculateAsymmetry
I got similar results and they were within the error value of the average. |
KarlPalmen
added a commit
that referenced
this pull request
Oct 20, 2015
…tion_in_MuonCalculateAsymmetry Fix period subtraction in MuonCalculateAsymmetry
KarlPalmen
deleted the
13952_Fix_period_subtraction_in_MuonCalculateAsymmetry
branch
October 20, 2015 14:24
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Muon
Issues and pull requests related to muons
Patch Candidate
Urgent issues that must be included in a patch following a release
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.
Fixes #13952
MuonCalculateAsymmetry is used in the MuonAnalysis interface. It is a workflow algorithm that can take two input workspaces and combine them to calculate the asymmetry. There are two possible operations on the input workspaces: period addition (+) and period subtraction (-). In the first case, the input workspaces must be summed first and then the asymmetry must be calculated. In the latter case, the asymmetry must be calculated first and then the workspaces must be subtracted.
Before #13102, the period addition (+) was not working as expected, as the asymmetry was calculated before adding the counts. When fixing this, I broke the period subtraction (-), as I was not aware that the two operations are not equivalent. This PR is to fix the "-" operation I broke in #13102.
For tester:
There are two things to test here: period addition and period subtraction.
As an example, my fitting results are:
you should get similar results.
Release notes: http://www.mantidproject.org/index.php?title=Release_Notes_3_6_MuonAnalysis&diff=25401&oldid=25396