Necessary Maintenance #24601
Scintillation visibility by semi-analytic parametrization not supported for multi-cryostat detectors
Status:
Accepted
Priority:
Normal
Assignee:
-
Category:
Simulation
Target version:
-
Start date:
07/06/2020
Due date:
% Done:
0%
Estimated time:
Experiment:
-
Description
It appears that in several places larsim/LegacyLArG4/OpFastScintillation.cxx code assumes a single cryostat.
I don't understand the code well enough to fix the issues (in fact, I don't even know if there are actual issues), but this can be a joint effort.
Related issues
History
#1 Updated by Gianluca Petrillo 6 months ago
- Related to Bug #24600: OpFastScintillation only propagates light in the first cryostats added
#2 Updated by Kyle Knoepfel 6 months ago
- Status changed from New to Feedback
- Subject changed from Scintillation visiblity by semi-analytic parametrization not supported for multi-cryostat detectors to Scintillation visibility by semi-analytic parametrization not supported for multi-cryostat detectors
Is this issue addressed by larsim PR (https://github.com/LArSoft/larsim/pull/30)?
#3 Updated by Gianluca Petrillo 6 months ago
It is not.
That pull request just attempts to restore the functionality of the other photon visibility mapping method, the lookup table.
The subject here is to support the semi-analytic model.
#4 Updated by Kyle Knoepfel 6 months ago
- Status changed from Feedback to Accepted
We will organize a meeting to discuss this issue.