Inconsistency of triggering peaks definition

[dachengx]

In Events, the triggering peaks are used to find the event window:

straxen/straxen/plugins/events/events.py

Line 133 in f765c10

_is_triggering = self._is_triggering(peaks)

.

But in EventBasics, we did not distinguish the triggering and non-triggering peaks when assigning main S1/2:

straxen/straxen/plugins/events/event_basics_vanilla.py

Line 224 in f765c10

split_peaks = strax.split_by_containment(peaks, events)

. I think we should not consider the non-triggering peaks as main S1/2 to make the events building consistent.