Skip to content

Commit 4130676

Browse files
fengyvoidmarc1uk
andauthored
Add EBPMT for PMT pairing in EventBuilding (#332)
* Add EBPMT for PMT pairing in EventBuilding * Modifications based on comments from PR#332: 1. I move the get and check of `RWMRawWaveforms` and `BRFRawWaveforms` to line 75 to 85. 2. I move the check and return to line 44-49. 3. The line 122 was still at 122, but only print after the `[]` 4. For line 125 and 137, I change the usage of `[]` to a int AlmostCompleteWaveforms_CountHere, initialized to be zero. 5. I would want to use that but may want to leave this to future optimization... since everything is working right now. (not perfectly because I do see zero PMT waveform from some timestamps, but they can be removed in later steps no matter where they are from) 6. Now at line 339 to 341, changed to emplace as you suggest. * Fix a bug about NumWavesInCompleteSet. Previously, if NumWavesInCompleteSet was set to 140 and there are only ~133 PMT waveforms exist in data, the AlmostCompleteWaveforms[PMTCounterTimeNs] won't be created. But the usage of AlmostCompleteWaveforms[PMTCounterTimeNs] in Log will create an entry there. This won't affect the later code because there are checks to require AlmostCompleteWaveforms[PMTCounterTimeNs] > 5. Now the Log was removed and the tolerance for PMT number is changed to 10 at: ChannelKey.size() >= (NumWavesInCompleteSet - 10) * The correct file for previous commit --------- Co-authored-by: marc1uk <marc1uk_@hotmail.com>
1 parent b5fb3b5 commit 4130676

File tree

5 files changed

+637
-0
lines changed

5 files changed

+637
-0
lines changed

0 commit comments

Comments
 (0)