Skip to content
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

Bug-fix for s1 pattern likelihood cuts #137

Merged
merged 6 commits into from
Jan 28, 2018
Merged

Conversation

skazama
Copy link
Contributor

@skazama skazama commented Jan 24, 2018

Dan and Jelle found that dead PMTs are included in the calculation of expected hit-patterns (See this link), and then cuts are optimized again with bug-fix.

From this version, we decided to use s1 pattern likelihood cut for top and bottom PMT arrays separately as done in XENON100. This is because if we use combined one, information can be smeared due to many PMTs, and the combined variable might be no longer sensitive to rejecting bad events like AC.

Optimization of cuts are done in the same way as done in the combined variable (99% quantile for Rn220 SR1 data), but done as a function of s1(bottom) and s1(top), respectively.

Details can be seen in this note

@pdeperio
Copy link
Contributor

Thanks Shingo. Do you have waveforms of those few events that are removed by bottom in SR0?

@feigaodm are you able to show the effect of this new cut (and recent others) on AC model?

1 similar comment
@pdeperio
Copy link
Contributor

Thanks Shingo. Do you have waveforms of those few events that are removed by bottom in SR0?

@feigaodm are you able to show the effect of this new cut (and recent others) on AC model?

@skazama
Copy link
Contributor Author

skazama commented Jan 24, 2018

Waveform for rejected event can be seen here. If you carefully compare PMT hit pattern for s1 (bottom) and s2 (top), it is clear that s1 hits are clustered bottom left region, which should be top right region from s2 x-y position.

Copy link
Contributor

@pdeperio pdeperio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use ManyLichen to separate the two conditionals so we can easily see the value for both. For example, MuonVeto cut.

@pdeperio pdeperio merged commit 8b493fe into master Jan 28, 2018
@pdeperio pdeperio deleted the skazama-new-s1-pattern branch January 28, 2018 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants