Upgrades to support multi readout charge collection #39
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I had to update
TRestDetectorSignalToHitsProcess
so that it is capable to add charge to more than oneTRestDetectorReadoutPlane
. In general it will add charge to any readout plane definition where the hit is found inside the volume defined by cathode and readout plane positions (and the charge falls inside a readout channel).List of changes:
TRestDetectorSignalToHitsProcess
has now been fully documented. Including image showing the process input and output event. https://sultan.unizar.es/test/classTRestDetectorHitsToSignalProcess.htmlTRestDetectorReadout::GetHitsDaqChannelAtReadoutPlane
method added to check the daqId for a specific readout plane so thatTRestDetectorHitsToSignalProcess
iterates each plane and gets the signal id associated with each plane.TRestDetectorHitsEvent::DrawEvent
aesthetical improvements. And a fix.TRestDetectorHitsToSignalProcess
now returnsnullptr
if no signals have been found. The event will then be ignored.This update was required by segmented mesh microbulks.