Few fixes for logParser
Bug description
- logParser crashing when comment was given after
evgenConfig.nEventsPerJob
- logParser incorrectly printing a warning if CPU time between 1-8h and
nEventsPerJob=10000
- no check to make sure that actual number of events to be produced are among
nEventsAllowedInProduction
- this check was only done forCountHepMC
, so if someone would run with 10k events but then changeevgenConfig.nEventsPerJob=40000
in the jO file logParser would incorrectly succeed
Changes introduced
- When reading
nEventsPerJob
from jO file doline.split('=')[1].split('#')[0].strip()
- the 2nd bug is fixed by this replacement
- The 3rd bug is fixed by
Tests
Log files taken from ATLMCPROD-8408
Bug 1
Before fix:
After fix it finds the correct number of events
Bug 2
Also if we would set evgenConfig.nEventsPerJob=5000
we get an ERROR
as expected
Bug 3
Issues resolved
Closes #
Edited by Spyros Argyropoulos