larsoft v09_69_00
LArSoft v09_69_00 Release Notes
Download instructions for larsoft v09_69_00
Download instructions for just larsoftobj v09_28_01
Purpose
- approved PRs
New features
- LArSoft/larsim#111
- Refactoring of PhotonPropagation classes to substantially reduce overhead
- While inspecting the code for bugs I tend to make changes to the code to make tests. I've collected enough of these changes that it makes sense to put them together neatly.
- This PR improves performance times and hopefully improves the organisation of the code. The processing time per event doesn't change much, but the initialisation is substantially reduced.
- Functionality stays largely the same, but there is is a small change in the time propagation distribution, which is expected.
- Main changes:
- Change from using ROOT RNG to CLHEP::RandGeneral
- Create PhotonPropagationUtils to host functions defined twice elsewhere
- Use references of the fcl parameters, instead of copying the objects
- Move PropagationTimeModel and SemiAnalyticalModel to the phot namespace
- Simplify the initialisation and reduce the number of class members, moving several things around and creating some functions to use them during initialisation
- Round scintillation times instead of flooring them
- Stop using TVector3
- Regularise the syntax and clean comments
- LArSoft/larsim#112
- Save MotherTrackID in the SimPhotons
- Running the light simulation withSimPhotons we found the MotherTrackID attribute was not being filled. This PR fixes this.
- also add the MotherTrackID to PDFastSimPVS_module.cc
- LArSoft/larg4#46
- Changes in ParticleList.cc to run full light simulation
- Particles are now added to the particle list only on their first step. Also add particles to the particle list with energy below the energy cut if it is an optical photon.
- See the presentation
- LArSoft/larg4#44
- Currently simulated energy deposits for which we are not saving the relevant MCParticle follow the prescription of being stored with a trackID of:
-1 *the trackID of their first ancestor which was saved- This is applied in ParticleListAction, and the effects of it are applied to sim::SimEnergyDeposits in LArG4Detector.
- In SBND we use sim::SimEnergyDeposits for our TPC & PDS systems but sim::AuxDetHits for our CRT. I noticed that the same prescription is not applied to the sim::AuxDetHits to update their track IDs.
- This PR employs a similar block to do this. I haven't added this to the other objects mentioned in this area of the code but that may be something that people want to consider going forward.
Breaking Changes
- LArSoft/larsim#111 requires changes in some experiment code
Updated dependencies
- ifdhc v2_6_18
- ifdhc_config v2_6_18
- ifbeam v2_5_21
- nucondb v2_5_21
- ifdh_art v2_13_17
- nugen v1_17_07
- nusystematics v01_02_07
- ppfx v02_15_07
Change List
larsoft v09_69_00
- "changes since v09_68_00":v09_68_00...v09_69_00
lareventdisplay v09_08_08
- "changes since v09_08_07":LArSoft/lareventdisplay@v09_08_07...v09_08_08
larexamples v09_06_06
- "changes since v09_06_05":LArSoft/larexamples@v09_06_05...v09_06_06
larpandora v09_18_02
- "changes since v09_18_01":LArSoft/larpandora@v09_18_01...v09_18_02
larsimrad v09_06_12
- "changes since v09_06_11":LArSoft/larsimrad@v09_06_11...v09_06_12
larsimdnn v09_02_12
- "changes since v09_02_11":LArSoft/larsimdnn@v09_02_11...v09_02_12
larrecodnn v09_17_11
- "changes since v09_17_10":LArSoft/larrecodnn@v09_17_10...v09_17_11
larwirecell v09_11_02
larana v09_12_01
- "changes since v09_12_00":LArSoft/larana@v09_12_00...v09_12_01
larreco v09_17_02
- "changes since v09_17_01":LArSoft/larreco@v09_17_01...v09_17_02
larsim v09_32_00
- "changes since v09_31_05":LArSoft/larsim@v09_31_05...v09_32_00
larg4 v09_14_00
- "changes since v09_13_06":LArSoft/larg4@v09_13_06...v09_14_00