Skip to content

Move Maxime's improvements to EventBasics

Luca Scotto Lavina requested to merge upgrade_eventbasics into DonNabla-patch-1

Created by: JelleAalbers

This:

  • Upgrades EventBasics to include the extra fields suggested by Maxime in https://github.com/XENONnT/straxen/pull/65.
  • Adds the computation of the number of channels contributing to the secondary but not primary S1 in a separate plugin.
  • Modifies Maxime's event_info plugin to load the existing event_basics, but change the field name convention to the conventional one for double scatter analyses -- e.g. s1_xxx->s1_a_xxx, alt_s1_xxx -> s1_b_xxx.
    • I'm actually not sure if it is a good idea to maintain two conventions long-term. But at least this should allow you and others to test it and transition more smoothly, if you want.

The reason for separating the distinct channel computation is that this requires going back to relatively low-level information only available in peaks. An alternative would be to add e.g. does_channel_contribute bools for each channel to peak_basics, but that means peak_basics would become of a comparable size to peaks itself. Since we have ~500 PMTs, the per-channel info is the part of peaks that takes the most space.

@DonNabla Do you agree with this approach, and if so, could you test it on a Kr dataset? I only tried it on a one-minute background set. Feel free to make other changes as well: this PR is only to make a modification to your branch in https://github.com/XENONnT/straxen/pull/65.

Merge request reports