Remove check that CountHepMC == nEventsPerJob
Currently in the logParser
we check that CountHepMC == nEventsPerJob
where CountHepMC
is extracted from the following line:
11:06:58 CountHepMC INFO Events passing all checks and written = 2
and nEventsPerJob
is taken from
10:47:13 Py:Gen_tf INFO .nEventsPerJob = 200 # (Integer) number of input events per job. Possible values: value >= 0
I have a vague recollection that this helped to identify some cases for MG where the multiplier was not adjusting for the generation efficiency properly and the transform still managed to run, however I do not exactly remember the cases and I would not be able to reproduce it.
Since this check has been a source of some headaches recently we would propose to remove it unless someone thinks that this is necessary.
Tagging MG experts: @zmarshal @mcfayden @hmildner @svonbudd and @ewelina @lcorpe in case you want to discuss in the git meeting (also @fsiegert @cgutscho with who we discussed this already)