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

Unblind energy sidebands in cs1 #218

Closed
wants to merge 2 commits into from
Closed

Conversation

coderdj
Copy link
Contributor

@coderdj coderdj commented Mar 3, 2018

Once we're ready this should open up cs1<3 pe as well as 70<=cs1<80 pe. Unblinding low and high energy sidebands for WIMP search.

It's very debatable whether the high energy sideband is needed at all and I can't really think of a justification at the moment so we should discuss.

coderdj and others added 2 commits March 3, 2018 15:43
Since my default is to unblind everything that has potential signal at once (and no strong requests to unblind high-E)
@coderdj
Copy link
Contributor Author

coderdj commented Mar 4, 2018

👍 (I can't approve since I created)

Copy link
Member

@tunnell tunnell left a comment

Choose a reason for hiding this comment

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

Technically, good. Requires @pdeperio to merge I'd say, so nobody else please.

@tunnell tunnell requested a review from pdeperio March 4, 2018 13:12
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.

I made a commit as well, so technically I shouldn't merge too lol. But don't merge yet, I will use this branch locally.

@feigaodm
Copy link
Member

feigaodm commented Mar 4, 2018

What do we do if we see 10 events in 70-80 PE range?

@pdeperio
Copy link
Contributor

pdeperio commented Mar 4, 2018

We are just doing cs1 < 3 right now.

@feigaodm
Copy link
Member

feigaodm commented Mar 4, 2018

good, I didn't realize this new change. So 70-80 is not nessary at all.

Since this region won't be analyzed directly, do you think we can cut this if background model can not explain there at all? In my opinion it's ok as long as we don't cut anything below 70.

@feigaodm
Copy link
Member

feigaodm commented Mar 5, 2018

not necessary anymore PR #216

@feigaodm feigaodm closed this Mar 5, 2018
@feigaodm feigaodm deleted the sideband_unblinding_sr1 branch March 5, 2018 16:06
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.

None yet

4 participants