Herwig7_i: Herwig fix

In this MR we introduced a checker script that searches for command duplicates in the Herwig7 configuration files, this helps to validate changes and to make sure that no command is overwritten with a wrong value. The code contains a section to check on the number of some files. This check seems to also find other file from Powheg with a very similar name. Since there should anyways never be more than one config file we want to remove this, because this currently breaks the ART tests as you can see here, marking this as urgent, since this could break the production when propagated into a release and there is a release to be built as soon as possible when this is merged. The fix was validated for the problematic JOs:

Gen_tf.py '--ecmEnergy=13600' '--jobConfig=421363' '--maxEvents=100' '--outputEVNTFile=test_phh7_wwlvlv.EVNT.pool.root’

and

Gen_tf.py '--ecmEnergy=13600' '--jobConfig=600433' '--maxEvents=100' '--outputEVNTFile=test_powheg_zmumu.EVNT.pool.root’

added Generators bugfix urgent labels

added 1 commit

• e6542b42 - Fix the JO Checker

Compare with previous version

changed the description

marked this merge request as ready

changed title from Herwig fix to Herwig7_i: Herwig fix

Jenkins please retry a build

This merge request affects 1 package:

• Generators/Herwig7_i

This merge request affects 1 file:

• Generators/Herwig7_i/python/Herwig7JOChecker.py

added main review-pending-level-1 labels

The system determined that CI tests (with names matching "^CITest_Trigger.*$") are not needed for this code change. They are not run. This is not an indicator to restart the job.

CI Result SUCCESS (hash e6542b42)

	Athena	AthGeneration
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 77383]

The changes are straightforward and there are no issues with the CI. Approving from L1.

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit d18ae08d