Old G4 FHiCLs Incorreclty Change Electron Lifetime

[marcodeltutto]

Some old G4 fcl files still use services.DetectorPropertiesService.Electronlifetime to change electron lifetime. This should be changed with the WC fcl parameter instead. Probably, we should remove these fcls and then create new ones when needed.

Example:
https://github.com/SBNSoftware/sbndcode/blob/39927777a4aa6747754dd3e2c991016dc7246804/sbndcode/JobConfigurations/standard/g4/g4_0.75ms_electron_lifetime.fcl#L3C1-L3C52

[absolution1]

Hi @marcodeltutto
I think we need to pin wirecell and larsoft's electron lifetime to use a single value as there are ultimately multiple stages in the workflow that depend on the electron lifetime being set (some rely on wirecell and some rely on the detector properties service). Setting the value in a single place massively reduces failure modes

We've already done this in the dune far detector: https://github.com/DUNE/dunereco/blob/develop/dunereco/DUNEWireCell/wirecell_dune.fcl#L578

[marcodeltutto]

Yes, totally agree @absolution1

[bear-is-asleep]

Tagging PR #514 for reference to more discussion.