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

tuning clustering for SR1 #615

Merged
merged 8 commits into from
Oct 17, 2017
Merged

tuning clustering for SR1 #615

merged 8 commits into from
Oct 17, 2017

Conversation

feigaodm
Copy link
Member

@feigaodm feigaodm commented Oct 5, 2017

When evaluating pax efficiency to find S1 and S2s, Joey found that pax performance is not perfect at lowest energy, see Fig 8 of this note. By examining lots of simulated WFs, we conclude that there are three directions pax can further improve the ability to merge single electrons from S2s deep in the TPC.

1, NaturalBreaks clustering threshold is too low. See this plot based on FAX simulations.
screen shot 2017-10-05 at 3 23 25 pm
The original threshold to split the peak into too is too low. By simulation we see that the more than 1% of deep S2s can be splitted as two peaks. So we propose to increase the threshold to a solid line where we expect to improve pax's ability to merge S2 signals. This will probably affect our ability to distinguish multiple scatters, however, as this clustering is applied after the gap_size clustering, the impact of it is much smaller compared to gap_size clustering. This Fix the example WF in joey's note, Fig 10.

2, The gap size clustering threshold is too small (currently set as 2.5us). As shown in the WF below, at a smaller probability ~1%, gaps between deep S2s can be larger than 2.5 us, this is probably due to a larger diffusion constant at a lower field in SR1.
screen shot 2017-10-03 at 3 12 57 pm
So we'd like to fix this by increasing the gap size to 3.0us, by doing this, we are able to merge the S2s again and do not split deep S2s.
screen shot 2017-10-03 at 3 11 32 pm

3, After the first gap_size clustering to isolate S1 signals, sometimes single electron S2s from deep TPC can be splitted into several S2 clusters. There is a small probability that one (or more) single electron S2s will be mis-classified as S1 signal, and those hits will be removed when we construct S2s. See this WF as an example.
screen shot 2017-10-05 at 3 38 48 pm
In order to solve this problem, we apply almost the same classification algrithm here during the gap_size cluster algorithms. Then this problem is solved, as seen in the following WF.
screen shot 2017-10-05 at 3 41 04 pm

Problem 2 and problem 3 both cause ~1% efficiency loss to reconstruct S2s. While we noticed that this effect is small in acceptance, but could be important in terms of ER band leakage as real S2s are splitted.

@JosephJHowlett
Copy link
Contributor

I checked the acceptance of small S2s with this branch compared with several earlier pax versions. It seems to mostly compensate the problems introduced by our weaker field. After several iterations of Fei tuning and me rechecking, we decided that we shouldn't keep pushing the max_gap_size_in_cluster parameter to boost the acceptance ad-infinitum, since we may get worse at resolving S2s from multiple scatters.

screen shot 2017-10-17 at 11 53 14 am

For now we decided to merge and let MC people check the resolving power.

Copy link
Contributor

@JosephJHowlett JosephJHowlett left a comment

Choose a reason for hiding this comment

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

Shows clear improvement in S2 acceptance below 500pe. Fei's point (1) was necessary and point (3) was a bug. More work could still be done to improve upon point (2), but not without validating multiple-scatter rejection.

@feigaodm feigaodm merged commit f580b71 into master Oct 17, 2017
@feigaodm feigaodm deleted the clustering_natural_breaks_sr1 branch October 17, 2017 19:40
JelleAalbers pushed a commit that referenced this pull request Aug 7, 2018
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.

2 participants