Use C++20 Spans in BLS Vertexing code

std::span in C++20 allows an interface to take any contiguous container (std::vector or std::array) this allows greater flexibility in design allowing fewer allocations.

This merge request affects 5 packages:

• Event/xAOD/xAODBPhys
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys
• PhysicsAnalysis/JpsiUpsilonTools
• Tracking/TrkVertexFitter/TrkVKalVrtFitter
• Tracking/TrkVertexFitter/TrkVertexAnalysisUtils

Affected files list will not be printed in this case

Adding @wwalko ,@jcatmore ,@akraszna ,@abarton ,@bouhova ,@emmat as watchers

added Analysis Derivation EDM Tracking analysis-review-required main review-pending-level-1 labels

WARNING: big files (>100K) are found in the changeset

108K in file Tracking/TrkVertexFitter/TrkVertexAnalysisUtils/src/V0Tools.cxx

CI integration tests for projects Athena are cancelled because of compilation error(s)

CI Result FAILURE (hash 63a45dcc)

	Athena	AnalysisBase	AthAnalysis
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 1, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 9063] (remote access info)

added 1 commit

• ce1f11fd - Fix Monitoring Alg

Compare with previous version

This merge request affects 6 packages:

• Event/xAOD/xAODBPhys
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys
• PhysicsAnalysis/JpsiUpsilonTools
• Tracking/TrkVertexFitter/TrkVKalVrtFitter
• Tracking/TrkVertexFitter/TrkVertexAnalysisUtils
• Trigger/TrigMonitoring/TrigBphysMonitoring

Affected files list will not be printed in this case

Adding @emmat ,@akraszna ,@jcatmore ,@tursom ,@abarton ,@bouhova ,@wwalko as watchers

added DQ Trigger labels

WARNING: big files (>100K) are found in the changeset

108K in file Tracking/TrkVertexFitter/TrkVertexAnalysisUtils/src/V0Tools.cxx

marked this merge request as draft

CI Result SUCCESS (hash ce1f11fd)

	Athena	AnalysisBase	AthAnalysis
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
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 9072] (remote access info)

Switching tags while under discussion

--Qiyu, L1

removed review-pending-level-1 label

added review-user-action-required label

added 1 commit

• 09287120 - Address feedback

Compare with previous version

resolved all threads

marked this merge request as ready

This merge request affects 6 packages:

• Event/xAOD/xAODBPhys
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys
• PhysicsAnalysis/JpsiUpsilonTools
• Tracking/TrkVertexFitter/TrkVKalVrtFitter
• Tracking/TrkVertexFitter/TrkVertexAnalysisUtils
• Trigger/TrigMonitoring/TrigBphysMonitoring

Affected files list will not be printed in this case

Adding @emmat ,@tursom ,@wwalko ,@akraszna ,@abarton ,@bouhova ,@jcatmore as watchers

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

WARNING: big files (>100K) are found in the changeset

108K in file Tracking/TrkVertexFitter/TrkVertexAnalysisUtils/src/V0Tools.cxx

CI Result SUCCESS (hash 09287120)

	Athena	AnalysisBase	AthAnalysis
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
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 9120] (remote access info)

Only one comment on the indentations. Others looks good to me.

--Mingxu (L1)

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

Looks no impact on analysis, approving from analysis side. Tong (AR)

removed analysis-review-required label

added analysis-review-approved label

added 1 commit

• a0526a2b - fix indent

Compare with previous version

resolved all threads

This merge request affects 6 packages:

• Event/xAOD/xAODBPhys
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys
• PhysicsAnalysis/JpsiUpsilonTools
• Tracking/TrkVertexFitter/TrkVKalVrtFitter
• Tracking/TrkVertexFitter/TrkVertexAnalysisUtils
• Trigger/TrigMonitoring/TrigBphysMonitoring

Affected files list will not be printed in this case

Adding @bouhova ,@jcatmore ,@wwalko ,@tursom ,@akraszna ,@abarton ,@emmat as watchers

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

WARNING: big files (>100K) are found in the changeset

108K in file Tracking/TrkVertexFitter/TrkVertexAnalysisUtils/src/V0Tools.cxx

CI Result SUCCESS (hash a0526a2b)

	Athena	AnalysisBase	AthAnalysis
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
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 9144] (remote access info)

MR looks fine from L1 perspective, CI shows no problems - approving

Cheers, Qiyu, L1

removed review-pending-level-1 label

added review-approved label

merged

mentioned in commit 7755be4c