Follow-up from "Resolve "Check the energy dependence of the shower chi^2 calculation""
The following discussions from !359 (merged) should be addressed:
-
@akorneev started a discussion: (+1 comment) For
isDiMuFlag
I suggest to use env. variable to ease the command line manipulation, like this:string env(char* v) { const char* s = getenv(v); return s ? s : ""; } ... main() ... const bool needDiMuon = (env("NEEDDIMUON") == "1"); ...
-
@akorneev started a discussion: (+3 comments) Here, I am not sure on the logic of
(ecal1 < binErange[nEnergy - 1])
andisEle100
checks. It that needed at all? - as I can see the energies abovebinErange[nEnergy - 1]
will be truncated by histograms itself.Also, to simplify the
xxx->Fill()
branching code below, is it possible to always fill all three histograms? -
@akorneev started a discussion: is the check needed?