Add GN1 to derivation config

• Preparation to add GN1 to derivation config (this will be done in a new MR once the files are in the group area)
• Store track leptonID as char instead of int
• Changing ONNX output level from warning to error (at some point we should figure out how to remove the warnings about unused nodes)

There are leftover errors since the GN1 paths are still in the dev area. Perhaps @pgadow can move them to the group space?

@dguest @nkakati

This merge request affects 3 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkFlavourTag
• PhysicsAnalysis/JetTagging/FlavorTagDiscriminants
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging

Affected files list will not be printed in this case

Adding @nishu ,@rcreager ,@duperrin ,@sanmay ,@dguest ,@bdong ,@lidiaz as watchers

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

changed the description

Hi @svanstro,

two quick comments: -it is not allowed to point to /dev/ areas for trainings (you need to move them to the official areas), -you are not just changing the derivation config but also the config used at Reco level (which is not obvious from the description).

Cheers, Arnaud

changed the description

CI Result FAILURE (hash 6b835c22)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional 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
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 52037]

added review-user-action-required label and removed review-pending-level-1 label

mentioned in merge request atlas-flavor-tagging-tools/training-dataset-dumper!352 (merged)

added 1 commit

• 609fa1c9 - bugfix

Compare with previous version

This merge request affects 3 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkFlavourTag
• PhysicsAnalysis/JetTagging/FlavorTagDiscriminants
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging

Affected files list will not be printed in this case

Adding @nishu ,@rcreager ,@duperrin ,@sanmay ,@dguest ,@bdong ,@lidiaz as watchers

added review-pending-level-1 label and removed review-user-action-required label

removed analysis-review-required label

removed review-pending-level-1 label

added review-user-action-required label

added analysis-review-required label

CI Result FAILURE (hash 609fa1c9)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional 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
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 52134]

added 1 commit

• 08d756d8 - remove networks

Compare with previous version

This merge request affects 2 packages:

• PhysicsAnalysis/JetTagging/FlavorTagDiscriminants
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging

Affected files list will not be printed in this case

Adding @dguest ,@duperrin ,@lidiaz as watchers

added review-pending-level-1 label and removed review-user-action-required label