Skip to content

Muon reconstruction - Dump EMEO muons into seperate container

Johannes Junggeburth requested to merge jojungge/athena:commission_flags into master

Hi,

this is a follow-up MR of !48353 (merged) & !48090 (merged). Dumping the muons from the incomplete commissioning chain turned out to cause more confusion than I was anticipating and hence those are now dumped into a separate EMEO_Muons container. Further changes made in this MR:

  • Introduce a flag to steer the EMEO chain at one place (runCommissioning)
  • Migrate the changes made in RecExCommon to the ComponentAccumulator

Adding labels: bugfix & urgent as these changes are needed to launch the Run 3 CP samples Tagging: @gunal, @pamelaf, @mvanadia, @gabarone, @zmarshal, @emoyse, @elmsheus, @girolamo, @kado, @tlari, @jchapman

I quickly checked that the commissioning muons are no longer in the container. The ESD content before the fix:

root [2] BasicTesterTree->Scan("truth_muons_pt:truth_muons_eta:truth_muons_phi:truth_muons_truthOrigin:muons_pt:muons_eta:muons_phi:muons_allAuthors:muons_truthLink")
***********************************************************************************************************************************
*    Row   * Instance * truth_muo * truth_muo * truth_muo * truth_muo *  muons_pt * muons_eta * muons_phi * muons_all * muons_tru *
***********************************************************************************************************************************
*        0 *        0 * 72.103248 * -1.907535 * 2.2805099 *        -1 * 74.209281 * -1.907971 * 2.2805261 *       262 *         0 *
*        0 *        1 *           *           *           *           * 27.404964 * -1.943912 * -1.173512 *       518 *     65535 *
*        0 *        2 *           *           *           *           * 27.049877 * -1.941530 * -1.174170 *      4128 *     65535 *
*        0 *        3 *           *           *           *           * 57.315792 * -1.906756 * 2.2778554 *      4128 *         0 *
*        1 *        0 * 1.6845160 * 0.4235583 * 0.1485285 *        -1 * 6.7173919 * -0.793375 * -1.975716 *       512 *     65535 *
*        2 *        0 * 16.021539 * -0.494812 * -2.142692 *        -1 * 33.685775 * -0.038570 * 0.0342123 *       256 *         1 *
*        2 *        1 * 34.833126 * -0.038818 * 0.0342192 *        -1 * 16.411476 * -0.495370 * -2.142272 *         6 *         0 *
*        2 *        2 * 0.3243557 * 1.1361835 * -0.474551 *        -1 *           *           *           *           *           *
*        3 *        0 * 39.817718 * 0.5948042 * 2.3539297 *        -1 * 48.560047 * 0.8190641 * -1.072418 *       262 *         1 *
*        3 *        1 * 49.418952 * 0.8195796 * -1.072594 *        -1 * 39.727317 * 0.5950745 * 2.3539161 *       262 *         0 *
*        4 *        0 * 35.803924 * 2.3220346 * 1.6843719 *        -1 * 37.240909 * 2.3213911 * 1.6843382 *       262 *         0 *
*        4 *        1 *           *           *           *           * 11.237578 * 2.3220806 * 1.6896699 *        32 *         0 *
*        4 *        2 *           *           *           *           * 38.481311 * 2.3209693 * 1.6873409 *      4128 *         0 *

For each truth muon in the endcaps, there are two reconstructed muons in the container. One of them is having an allAuthor value beyond 4096, which marks the muon to originate from the commissioning chain. The content post-fix:

root [1] BasicTesterTree->Scan("truth_muons_pt:truth_muons_eta:truth_muons_phi:truth_muons_truthOrigin:muons_pt:muons_eta:muons_phi:muons_allAuthors:muons_truthLink")
***********************************************************************************************************************************
*    Row   * Instance * truth_muo * truth_muo * truth_muo * truth_muo *  muons_pt * muons_eta * muons_phi * muons_all * muons_tru *
***********************************************************************************************************************************
*        0 *        0 * 72.103248 * -1.907535 * 2.2805099 *        -1 * 74.209281 * -1.907971 * 2.2805261 *       262 *         0 *
*        0 *        1 *           *           *           *           * 27.404964 * -1.943912 * -1.173512 *       518 *     65535 *
*        1 *        0 * 16.021539 * -0.494812 * -2.142692 *        -1 * 33.685775 * -0.038570 * 0.0342123 *       256 *         1 *
*        1 *        1 * 34.833126 * -0.038818 * 0.0342192 *        -1 * 16.411476 * -0.495370 * -2.142272 *         6 *         0 *
*        1 *        2 * 0.3243557 * 1.1361835 * -0.474551 *        -1 *           *           *           *           *           *
*        2 *        0 * 39.817718 * 0.5948042 * 2.3539297 *        -1 * 48.560047 * 0.8190641 * -1.072418 *       262 *         1 *
*        2 *        1 * 49.418952 * 0.8195796 * -1.072594 *        -1 * 39.727317 * 0.5950745 * 2.3539161 *       262 *         0 *
*        3 *        0 * 48.927455 * -0.598788 * -2.113163 *        -1 * 48.203460 * -0.599041 * -2.113245 *       262 *         0 *
*        3 *        1 * 28.122707 * -1.692324 * 1.1618440 *        -1 * 26.889921 * -1.692103 * 1.1622408 *       262 *         1 *
*        4 *        0 * 1.6845160 * 0.4235583 * 0.1485285 *        -1 * 6.7173919 * -0.793375 * -1.975716 *       512 *     65535 *
*        5 *        0 * 6.5165147 * -2.382118 * 2.2902853 *        -1 * 6.2814431 * -2.379925 * 2.2899332 *       262 *         0 *
*        6 *        0 *           *           *           *           *           *           *           *           *           *
*        7 *        0 * 23.247673 * -1.430199 * 0.1711541 *        -1 * 23.340683 * -1.430033 * 0.1711994 *       262 *         0 *
*        8 *        0 * 38.290687 * 2.2749426 * 1.8196598 *        -1 * 37.778312 * 2.2749512 * 1.8197293 *       262 *         0 *
*        8 *        1 * 37.965843 * 1.7961397 * -1.144369 *        -1 * 38.137207 * 1.7964432 * -1.144508 *       262 *         1 *
*        8 *        2 * 0.6109579 * 1.4702705 * 2.8308053 *        -1 *           *           *           *           *           *

the muons with the commissioning author bit are successfully removed. To speed up the validation, I submitted a larger scale sample to the grid: https://bigpanda.cern.ch/task/27902929 However, I am afraid that the group quota could be quite low here.

Reference sample for the comparison from last week:

ESD:  group.det-muon.ParticleGun_DiMu_Pt10to100.ESD.Run3_2NSW_160122a_reco_22050_EXT0 (https://bigpanda.cern.ch/task/27853056/)
AOD:  group.det-muon.ParticleGun_DiMu_Pt10to100.ESD.Run3_2NSW_160122a_reco_22050_EXT1 (https://bigpanda.cern.ch/task/27853056/)

Dump of the Track & Muon containers in a local test ESD:

                    xAOD::MuonContainer | EMEO_Muons, HLT_MuonsCB_FS, HLT_MuonsCB_LRT, HLT_MuonsCB_RoI, HLT_MuonsCB_RoI_TLA, HLT_MuonsIso, HLT_Muons_FS, HLT_Muons_RoI, Muons, MuonsLRT, Staus
                  xAOD::MuonAuxContainer | EMEO_MuonsAux., HLT_MuonsCB_FSAux., HLT_MuonsCB_LRTAux., HLT_MuonsCB_RoIAux., HLT_MuonsCB_RoI_TLAAux., HLT_MuonsIsoAux., HLT_Muons_FSAux., HLT_Muons_RoIAux., MuonsAux., MuonsLRTAux., StausAux.
         xAOD::TrackParticleAuxContainer | CombinedMuonTrackParticlesAux., CombinedMuonsLRTTrackParticlesAux., CombinedStauTrackParticlesAux., EMEO_CombinedMuonTrackParticlesAux., EMEO_MuonSpectrometerTrackParticlesAux., ExtraPolatedMuonsLRTTrackParticlesAux., ExtrapolatedMuonTrackParticlesAux., ExtrapolatedStauTrackParticlesAux., GSFTrackParticlesAux., HLT_CBCombinedMuon_FSTrackParticlesAux., HLT_CBCombinedMuon_RoITrackParticlesAux., HLT_IDTrack_Bjet_FTFAux., HLT_IDTrack_Bjet_IDTrigAux., HLT_IDTrack_Bmumux_FTFAux., HLT_IDTrack_Bmumux_IDTrigAux., HLT_IDTrack_Cosmic_EFIDAux., HLT_IDTrack_Cosmic_FTFAux., HLT_IDTrack_Cosmic_IDTrigAux., HLT_IDTrack_ElecLRT_FTFAux., HLT_IDTrack_ElecLRT_IDTrigAux., HLT_IDTrack_Electron_FTFAux., HLT_IDTrack_Electron_GSFAux., HLT_IDTrack_Electron_IDTrigAux., HLT_IDTrack_FSLRT_FTFAux., HLT_IDTrack_FSLRT_IDTrigAux., HLT_IDTrack_FS_FTFAux., HLT_IDTrack_JetSuper_FTFAux., HLT_IDTrack_MinBias_IDTrigAux., HLT_IDTrack_MuonIso_FTFAux., HLT_IDTrack_MuonIso_IDTrigAux., HLT_IDTrack_MuonLRT_FTFAux., HLT_IDTrack_MuonLRT_IDTrigAux., HLT_IDTrack_Muon_FTFAux., HLT_IDTrack_Muon_IDTrigAux., HLT_IDTrack_TauCore_FTFAux., HLT_IDTrack_TauIso_FTFAux., HLT_IDTrack_Tau_FTFAux., HLT_IDTrack_Tau_IDTrigAux., HLT_MSExtrapolatedMuons_FSTrackParticlesAux., HLT_MSExtrapolatedMuons_RoITrackParticlesAux., HLT_MSOnlyExtrapolatedMuons_FSTrackParticlesAux., InDetDisappearingTrackParticlesAux., InDetForwardTrackParticlesAux., InDetLargeD0TrackParticlesAux., InDetTrackParticlesAux., LRTGSFTrackParticlesAux., MSOnlyExtraPolatedMuonsLRTTrackParticlesAux., MSOnlyExtrapolatedMuonTrackParticlesAux., MSonlyTrackletsAux., MuonSpectrometerTrackParticlesAux.
            xAOD::TrackParticleContainer | CombinedMuonTrackParticles, CombinedMuonsLRTTrackParticles, CombinedStauTrackParticles, EMEO_CombinedMuonTrackParticles, EMEO_MuonSpectrometerTrackParticles, ExtraPolatedMuonsLRTTrackParticles, ExtrapolatedMuonTrackParticles, ExtrapolatedStauTrackParticles, GSFTrackParticles, HLT_CBCombinedMuon_FSTrackParticles, HLT_CBCombinedMuon_RoITrackParticles, HLT_IDTrack_Bjet_FTF, HLT_IDTrack_Bjet_IDTrig, HLT_IDTrack_Bmumux_FTF, HLT_IDTrack_Bmumux_IDTrig, HLT_IDTrack_Cosmic_EFID, HLT_IDTrack_Cosmic_FTF, HLT_IDTrack_Cosmic_IDTrig, HLT_IDTrack_ElecLRT_FTF, HLT_IDTrack_ElecLRT_IDTrig, HLT_IDTrack_Electron_FTF, HLT_IDTrack_Electron_GSF, HLT_IDTrack_Electron_IDTrig, HLT_IDTrack_FSLRT_FTF, HLT_IDTrack_FSLRT_IDTrig, HLT_IDTrack_FS_FTF, HLT_IDTrack_JetSuper_FTF, HLT_IDTrack_MinBias_IDTrig, HLT_IDTrack_MuonIso_FTF, HLT_IDTrack_MuonIso_IDTrig, HLT_IDTrack_MuonLRT_FTF, HLT_IDTrack_MuonLRT_IDTrig, HLT_IDTrack_Muon_FTF, HLT_IDTrack_Muon_IDTrig, HLT_IDTrack_TauCore_FTF, HLT_IDTrack_TauIso_FTF, HLT_IDTrack_Tau_FTF, HLT_IDTrack_Tau_IDTrig, HLT_MSExtrapolatedMuons_FSTrackParticles, HLT_MSExtrapolatedMuons_RoITrackParticles, HLT_MSOnlyExtrapolatedMuons_FSTrackParticles, InDetDisappearingTrackParticles, InDetForwardTrackParticles, InDetLargeD0TrackParticles, InDetTrackParticles, LRTGSFTrackParticles, MSOnlyExtraPolatedMuonsLRTTrackParticles, MSOnlyExtrapolatedMuonTrackParticles, MSonlyTracklets, MuonSpectrometerTrackParticles
Edited by Johannes Junggeburth

Merge request reports