BTagging config flags creation

Create first version of BTaggingConfigFlags.py to be used in new configuration files.

A patch proposed in !25465 (merged) is implemented. @wlampl added for comments. It solved two issues for my BTagRun3Config.py script:

• with 2 threads, OutputStreamESD was frequently in ERROR state
• when the job were successful some variables were missing in the output

This merge request affects 2 packages:

• Control/AthenaConfiguration
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging

Adding @dguest ,@guirriec ,@ssnyder ,@rbianchi ,@lidiaz as watchers

added Analysis BTagging Core Derivation JetEtmiss master review-pending-level-1 labels

added NewConfig label

CI Result SUCCESS

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-26397-2019-09-06-19-36
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 3051]

added 1 commit

• 64480f88 - New prefix passed to SecVertexing

Compare with previous version

This merge request affects 2 packages:

• Control/AthenaConfiguration
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging

Adding @dguest ,@guirriec ,@ssnyder ,@rbianchi ,@lidiaz as watchers

CI Result FAILURE

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-26397-2019-09-09-15-53
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 3095]

Hi

It seems that the error is not due to my last commit. Please restart if you think that is is worth to do it.

Manu

added 1 commit

• 0ecc1bcd - Add more flags in BTaggingConfigFlags.py

Compare with previous version

This merge request affects 3 packages:

• Control/AthenaConfiguration
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging
• PhysicsAnalysis/JetTagging/JetTagTools

Adding @dguest ,@guirriec ,@ssnyder ,@rbianchi ,@lidiaz as watchers

Jenkins please retry a build

CI Result SUCCESS

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-26397-2019-09-11-00-51
Athena: number of compilation errors 0, warnings 2
AthSimulation: number of compilation errors 0, warnings 1
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 3149]

Hi all,

just to say that I am happy with the little patch to ComponentAccumulator.py that is part of this MR.

• Walter

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

I checked the changes and it looks good to me. But I'd like to ask L2 to have a quick look since it's not only simple changes IMO.--MR L1

Agree, quite extensive changes, but I also do not see anything problematic, therefore approve.

Pavol [as L2 MR shifter]

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

merged

mentioned in commit 231ed5fc

added sweep:ignore label