Clean up unused vertex collections in JetBTaggingAlg

While investigating the impact of ACTS on FTAG secondary vertexing, I noticed that the secondary vertex collections were retrieved in the JetBTaggingAlg but not used. They are instead used in the BTagLightSecVertexing tool. They are consequently removed from the JetBTaggingAlg. Run 3 derivation outputs have been checked to be unchanged.

FYI @ligang @bdong

added BTagging label

This merge request affects 1 package:

• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging

This merge request affects 3 files:

• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging/BTagging/JetBTaggingAlg.h
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging/python/JetBTaggingAlgConfig.py
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging/src/JetBTaggingAlg.cxx

Adding @dguest ,@bdong ,@lidiaz as watchers

added 23.0 Analysis Derivation JetEtmiss review-pending-level-1 labels

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

CI Result SUCCESS (hash e08f2c41)

	Athena
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 73513]

resolved all threads

CI clean and changes look good from an L1 point of view. Approving - L1

removed review-pending-level-1 label

added review-approved label

added review-approved-tier0 label

added review-approved-point1 label

Hi @tstreble,

unfortunately, this MR has picked up a merge conflict. Please resolve!

Sorry for the inconvenience, Walter

added review-user-action-required label

added 43 commits

• e08f2c41...262f4b05 - 42 commits from branch atlas:23.0
• 59a41d5e - Clean up unused vertex collections in JetBTaggingAlg

Compare with previous version

This merge request affects 1 package:

• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging

This merge request affects 3 files:

• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging/BTagging/JetBTaggingAlg.h
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging/python/JetBTaggingAlgConfig.py
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging/src/JetBTaggingAlg.cxx

Adding @dguest ,@bdong ,@lidiaz as watchers

added review-pending-level-1 label and removed review-approved review-approved-point1 review-approved-tier0 review-user-action-required labels

resolved all threads

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

CI Result SUCCESS (hash 59a41d5e)

	Athena
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 73645]