Project

General

Profile

Necessary Maintenance #24601

Scintillation visibility by semi-analytic parametrization not supported for multi-cryostat detectors

Added by Gianluca Petrillo about 1 month ago. Updated 10 days ago.

Status:
Accepted
Priority:
Normal
Assignee:
-
Category:
Simulation
Target version:
-
Start date:
07/06/2020
Due date:
% Done:

0%

Estimated time:
Experiment:
-
Duration:

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

Related to LArSoft - Bug #24600: OpFastScintillation only propagates light in the first cryostatsClosed07/06/2020

History

#1 Updated by Gianluca Petrillo about 1 month ago

  • Related to Bug #24600: OpFastScintillation only propagates light in the first cryostats added

#2 Updated by Kyle Knoepfel 24 days 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 24 days 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 10 days ago

  • Status changed from Feedback to Accepted

We will organize a meeting to discuss this issue.



Also available in: Atom PDF