Necessary Maintenance #11468
Clean larsim/Simulation/SimPhotons.h from DUNE code
larsim/Simulation/SimPhotons.h (line 73 in LArSoft
v05_00_00_rc2) contains a comment:
/// \todo: Remove this class when DUNE makes the next round of production
/// MC files - after 11 September 2013 brebel
(note that the automatic LBNE -> DUNE renaming did work...)
DUNE should state if that is still needed. If so, a better solution needs to be devised.
#3 Updated by Alexander Himmel about 5 years ago
Tingjun Yang wrote:
Alex should comment on this.
OK, so the DUNE10kT photons class was an early attempt at simulating the full sized DUNE (then LBNE) detector written by Zepeng. It was needed because the libraries just weren't working for the full-sized detector. This is not in use, but that is because we simply do not simulate photons in the full-sized detector yet. So, I guess I wouldn't object to removing this since it hasn't been used since then but we don't have "something better" to replace it with at this stage, and probably won't for some time.
#5 Updated by Katherine Lato over 4 years ago
From an email conversation on 9/29/16, the plan is to assign this to someone who is already working near the code, but not until the current task is completed. (Alex will give the name in a month.) Also will close the duplicated issue, #10741.
Alex noted that need to remove the "Parameterized photon transport" that fills that class, not sure what it will break so some investigation is required. On the plus side, removing the code will avoid the confusion that occurred recently by a student working on implementing a backtracker for the photon simulation.