Add muons built from large d0 tracking.

Hi everybody,

this merge request is keeping track of the implementation of the largeD0 tracks into the muon reconstruction. At the moment everything is a bit hacky and hence let's keep the MR just WIP. I'll tide up the scripts once the final setup is running.

Adding: @goblirsc, @sroe, @nkoehler, @jstupak, @mdanning, @npetters, @emoyse, @szambito, @gartoni, @mvanadia and our well-admired friend the @atlasbot

P.S: Who's that famous guy being the philosopher of the day:

changed the description

Hi, also adding @emoyse

@jojungge, does this setup run already?

That setup runs but does not write the containers. I fixed that locally and will commit later. I am refining the naming flags etc.

changed the description

added 1 commit

• 35e7fd66 - Fix the physics validation for Large D0

Compare with previous version

added 134 commits

• 35e7fd66...d650c94d - 132 commits from branch atlas:master
• aa803a83 - Merge branch 'master' of ssh://gitlab.cern.ch:7999/jojungge/athena into large_d0_muons
• 711896bc - Add large d0 tracks to physics validation

Compare with previous version

added 1 commit

• 9a8325c7 - Write container output for muons

Compare with previous version

added 217 commits

• 9a8325c7...be7118a9 - 215 commits from branch atlas:master
• a1e3560f - Add d0 vs z0 validation histogram. Fix z0 sin theta bug
• eaed608d - Merge branch 'master' of ssh://gitlab.cern.ch:7999/jojungge/athena into large_d0_muons

Compare with previous version

added 64 commits

• eaed608d...e739c48f - 63 commits from branch atlas:master
• 6f25df85 - Merge branch 'master' of ssh://gitlab.cern.ch:7999/jojungge/athena into large_d0_muons

Compare with previous version

added 1 commit

• 5c338ff2 - Shovel LRT muons without associated ID track

Compare with previous version

added 1 commit

• a618e84d - Hupps that change was meant to be kept locally

Compare with previous version

added 43 commits

• a618e84d...6effd81e - 42 commits from branch atlas:master
• 8c261d47 - Merge branch 'master' of ssh://gitlab.cern.ch:7999/jojungge/athena into large_d0_muons

Compare with previous version

added 2 commits

• d6a645a2 - Merge branch 'master' of ssh://gitlab.cern.ch:7999/jojungge/athena into large_d0_muons
• b4b47b43 - Merge branch 'large_d0_muons' of ssh://gitlab.cern.ch:7999/jojungge/athena into large_d0_muons

Compare with previous version

added 327 commits

• b4b47b43...d8ffe7e2 - 324 commits from branch atlas:master
• e1811dd7 - Fix merge conflicts
• 57bc5a87 - Merge branch 'master' of ssh://gitlab.cern.ch:7999/jojungge/athena into large_d0_muons
• eb6d8151 - Put back the large radius tracking conifg'

Compare with previous version

added 114 commits

• eb6d8151...f58c75cf - 112 commits from branch atlas:master
• 2adb2afa - Merge branch 'master' of ssh://gitlab.cern.ch:7999/jojungge/athena into large_d0_muons
• 21d8b358 - Rename some of the containers

Compare with previous version

Dear @jojungge, I am very sorry for missing the Philosoper Of The Day! And you even thought to post this famous Alsacian theologist on his very birthday!

changed the description

added 1 commit

• b6b688b1 - Add the muon track particle containers for now

Compare with previous version

added 25 commits

• b6b688b1...e06b270b - 24 commits from branch atlas:master
• 971a4842 - Merge branch 'master' of ssh://gitlab.cern.ch:7999/jojungge/athena into large_d0_muons

Compare with previous version

added 3 commits

• 971a4842...63f5db57 - 2 commits from branch atlas:master
• b7e51c19 - Merge branch 'master' of ssh://gitlab.cern.ch:7999/jojungge/athena into large_d0_muons

Compare with previous version

added 222 commits

• b7e51c19...28857a9f - 221 commits from branch atlas:master
• eabfb14d - Fix conflict

Compare with previous version

added 381 commits

• eabfb14d...9386e960 - 378 commits from branch atlas:master
• 1848e8b1 - Merge branch 'master' of ssh://gitlab.cern.ch:7999/jojungge/athena into large_d0_muons
• 3b44fe45 - Merge branch 'master' of ssh://gitlab.cern.ch:7999/jojungge/athena into large_d0_muons
• edd8e32a - Remove IP cuts for LRT:

Compare with previous version

added 118 commits

• edd8e32a...20e9b098 - 117 commits from branch atlas:master
• 8f89fe8b - Merge branch 'master' into 'large_d0_muons'

Compare with previous version

unmarked as a Work In Progress

changed the description

marked as a Work In Progress

This merge request affects 6 packages:

• InnerDetector/InDetExample/InDetRecExample
• InnerDetector/InDetValidation/InDetPhysValMonitoring
• PhysicsAnalysis/PhysicsValidation/PhysValMonitoring
• Reconstruction/MuonIdentification/MuonCombinedAlgs
• Reconstruction/MuonIdentification/MuonCombinedBaseTools
• Reconstruction/MuonIdentification/MuonCombinedRecExample

Adding @goetz ,@rosati ,@wleight ,@sroe ,@nkoehler as watchers

added Analysis InnerDetector Reconstruction master review-pending-level-1 labels

resolved all threads

Does anyone know where the truth-matching of the muons is actually performed? Thus far the information stored in the xAOD::Muon is pretty useless. The ID track is fine

CI Result SUCCESS (hash 8f89fe8b)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 21081]

added 1 commit

• 8e1415a4 - Fix validation jobs

Compare with previous version

WIP, so reverting the labels to the user.

Michele (L1 reviewer)

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

added Run2-DataReco-output-changed label

added 387 commits

• 8e1415a4...0f40f13d - 386 commits from branch atlas:master
• 1977bd9f - Merge branch 'master' of ssh://gitlab.cern.ch:7999/jojungge/athena into large_d0_muons

Compare with previous version

added 1 commit

• 348e1459 - Migrate the tool setup to MuonCombineAlgs and MuonCombinedTools. What a...

Compare with previous version

unmarked as a Work In Progress

This merge request affects 6 packages:

• InnerDetector/InDetExample/InDetRecExample
• InnerDetector/InDetValidation/InDetPhysValMonitoring
• PhysicsAnalysis/PhysicsValidation/PhysValMonitoring
• Reconstruction/MuonIdentification/MuonCombinedAlgs
• Reconstruction/MuonIdentification/MuonCombinedBaseTools
• Reconstruction/MuonIdentification/MuonCombinedRecExample

Adding @goetz ,@rosati ,@wleight ,@sroe ,@nkoehler as watchers

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

resolved all threads

First results have been presented in last week's MCP meeting: https://indico.cern.ch/event/960232/

If everyone agrees, I'd propose to merge this in and fine tune the remaining issues in a follow-up MR.

resolved all threads

added 1 commit

• c10b8957 - Revert change

Compare with previous version

This merge request affects 6 packages:

• InnerDetector/InDetExample/InDetRecExample
• InnerDetector/InDetValidation/InDetPhysValMonitoring
• PhysicsAnalysis/PhysicsValidation/PhysValMonitoring
• Reconstruction/MuonIdentification/MuonCombinedAlgs
• Reconstruction/MuonIdentification/MuonCombinedBaseTools
• Reconstruction/MuonIdentification/MuonCombinedRecExample

Adding @goetz ,@rosati ,@wleight ,@sroe ,@nkoehler as watchers

CI Result SUCCESS (hash 348e1459)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 21441]

CI Result SUCCESS (hash c10b8957)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 21443]

CI looks good, approving.

Cheers, Volker (L1)