classification update #576
Merged
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
MAVJ
approved these changes
May 31, 2017
feigaodm
added a commit
to XENON1T/lax
that referenced
this pull request
Jun 21, 2017
redundant since we added it in pax (XENON1T/pax#576)
|
can you include more documentation (note or plot) showing the tuning you did here? |
tunnell
pushed a commit
to XENON1T/lax
that referenced
this pull request
Jun 23, 2017
* Created Lichen for Scienceun1. Updated AmBe FV. Created NG FV * Removed repeated definition * Fix for codacy errors This should fix the style errors found here: https://www.codacy.com/app/weiyuehuan/lax/pullRequest?prid=694964 * Another codacy fix Fix for remaining 3 errors here: https://www.codacy.com/app/tunnell/lax/pullRequest?prid=694950 . Hope this will fix all * Update sciencerun1.py Corrected wrong dependency of FiducialCylinder1T cut * Update SR0 AmBe to correct values * Update sciencerun1.py * remove SingleElectronS2 cut redundant since we added it in pax (XENON1T/pax#576) * fix style * Fix trailing whitespace * Fix more lint whitespace issues
|
That is how the boundary looks in PaxV6.6.5 (same procedure explained and used in https://xecluster.lngs.infn.it/dokuwiki/doku.php?id=xenon:xenon1t:analysis:firstresults:exploring_se_cut) |
|
This should be compared to "Cut v4_RvsAft" in Figure 2 of your note (which was used in SR0 lax)? But then they look very similar, i.e. shouldn't the curve drop from AFT 0.4 to 0.5 according to this PR? Can you overlay the two? And then how about a plot of the change in |
|
@jhowl01 just pointed out that this has changed again in pax_v6.8.0: Can we please get more documentation on this, i.e. the overlay plot requested above? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Mis-identification of single electron S2 signals as S1 signal is a big contribution for the Lone-S1 rate in the detector, and significantly affect the accidental coincidence background.
A dedicated cut was defined by @MAVJ to reduce the probability of mis-identification. Instead of putting this cut in lax stage, we'd like to modify the peak classification in pax stage instead, which motivates this PR.
A minor modification of original cut is done to have slightly better acceptance for high AFT S1s (events happening on the top part of the TPC). Efficiency of this cut should be estimated together with other pax efficiency loss, such as this note