Kr83mcut

[ershockley]

This PR updates the MisIdS1SingleScatter cut, specifically for SR2 but also for SR1.

I'm not sure what the best way to organize this is so if anyone thinks we should not add this cut to e.g. sciencerun0.py or sciencerun1.py please let me know.

The note for the update can be found here. The cut is improved for both SR1 and SR2, where pax v6.10.1 data was used.

I also compared the cut with the similar KryptonMisIdS1, where I found that the latter removes a subset of those removed by MisIdS1SingleScatter, but doesn't remove all the events that it should. I think we should merge these two cuts, but also wanted to keep some type of backwards compatibility, so I just set the cuts equal in the sciencerun1 and sciencerun2 files. I left sciencerun0.py in case anyone wants to use the old cut.

Finally, this PR removes the MisIdS1SingleScatter cut previously defined in postsr1.py. Again, please let me know if a different organization is preferred.

[JelleAalbers]

Hi Evan, I have a conference this week, but perhaps someone else is available to review this. Otherwise I could do so next week (someone may have to remind me :-))

[JelleAalbers]

Hi Evan, thanks for this change. We currently only put cuts that were used for the frozen tonne-year paper analysis in the sciencerun0 and sciencerun1 lichen files. See the 'code organization' section of the readme on the main page.

So I think the MisIdS1SingleScatter cut was fine were it was (postsr1, since it applies to SR1 but is from a post-tonne-year analysis) and your new cut can be there as well, if it applies to SR1 and SR2. Then you can reference it in sciencerun2.py. Alternatively you could define it in sciencerun2 and then reference it in postsr1; up to you.