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

Trigger should not store events containing a veto #605

Open
coderdj opened this issue Aug 30, 2017 · 3 comments
Open

Trigger should not store events containing a veto #605

coderdj opened this issue Aug 30, 2017 · 3 comments

Comments

@coderdj
Copy link
Contributor

coderdj commented Aug 30, 2017

Specifically a BUSY or HE veto. Since our veto logic operates on large S2s, even if the whole S2 is blocked the trigger will presumably still trigger on the large S1. These events are excluded from every analysis and I can't think of a reason to keep them.

Of course if someone thinks of a reason why we want to keep these events it could be discussed.

@sanderbreur
Copy link
Contributor

sanderbreur commented Aug 30, 2017 via email

@coderdj
Copy link
Contributor Author

coderdj commented Aug 30, 2017

I don't really buy your physics case. We have other handles on the light yield and we've seen the Rn220 spectrum. Are there physics inputs derived from these numbers? If so I would be a little careful. I'm not sure how efficient the HE veto is at telling a large S1 from an S2* so your distributions could be biased in the case that certain S1s get vetoed and others don't. At least we'd have to check it.

But even if we decide to allow HE-vetoed events through in Rn220 we could still include the logic to drop them in neutron generator data, for example.

In both cases we have a configurable prescale that allows us to record a subset of full, unbiased events.

  • Edit: it works on width but we're dealing with an analog sum waveform that gets saturated and distorted at various levels (lastly in the fans) so not clear that it's infallible

@JelleAalbers
Copy link
Contributor

JelleAalbers commented Sep 18, 2017

Besides S1-analyses these events are used by the S2/tdiff cut (assuming at least some of the S2 is saved some of the time). Are there any estimates on the amount of data reduction that removing them would achieve?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants