search

XENONnT / straxen

Streaming analysis for XENON
BSD 3-Clause "New" or "Revised" License
20 stars 32 forks source link

Definition of `cs2_wo_timecorr` #1257

Closed dachengx closed 2 months ago

dachengx commented 1 year ago

The master branch currently says that cs2_wo_timecorr is

f'Corrected area of {peak_name} S2 before SEG/EE and elife corrections (s2 xy correction applied) [PE]'

https://github.com/XENONnT/straxen/blob/2e9eaaa7e146e8a19573849293aa796ac46630ee/straxen/plugins/events/corrected_areas.py#L201

But in the code(which was introduced from https://github.com/XENONnT/straxen/commit/388ce3a42bb78e70c2d09cf3989b3ec084ed7a73#diff-75af39866372953e59e003b61a91bed2bee9effbb8159dd6fc911bcd8e2142d6R738-R739), it is after elife correction applied.

dachengx commented 1 year ago

I would change the code, but not the descriptions at the following PR.

dachengx commented 1 year ago

PR: https://github.com/XENONnT/straxen/pull/1258

dachengx commented 2 months ago

MISSION ACCOMPLISHED by https://github.com/XENONnT/straxen/pull/1260