First version of Muon geometry and conditions compatible with dd4hep
Part of set of MRs : !172 (merged) LHCb!3511 (merged) Allen!822 (merged)
dd4hep Muon geometry and readout first version. Non all functions have been impemented yet
Changes the meaning of the Muon TileID (i.e. LHCbID) bits!
Merge request reports
Activity
added lhcb-dd4hep lhcb-gaussino labels
- Resolved by Sebastien Ponce
/ci-test
- [2022-04-08 00:04] Automatic merge failed in [lhcb-dd4hep#1091](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-dd4hep/1091/Detector/checkout
- [2022-04-08 00:18] Automatic merge failed in [lhcb-gaussino-new-cmake#134](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-gaussino-new-cmake/134/Detector/checkout
- [2022-04-08 00:18] Automatic merge failed in [lhcb-gaussino-fastsim#251](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-gaussino-fastsim/251/Detector/checkout
- [2022-04-08 00:19] Automatic merge failed in [lhcb-gaussino#1196](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-gaussino/1196/Detector/checkout
- [2022-04-08 21:21] Validation started with lhcb-master-mr#4205
- [2022-04-09 00:04] Validation started with lhcb-dd4hep#1092
- [2022-04-09 00:07] Validation started with lhcb-gaussino-fastsim#252
- [2022-04-09 00:07] Validation started with lhcb-gaussino-new-cmake#135
- [2022-04-09 00:08] Validation started with lhcb-gaussino#1197
- [2022-04-12 00:03] Validation started with lhcb-dd4hep#1093
- [2022-04-12 00:14] Validation started with lhcb-gaussino-fastsim#253
- [2022-04-12 00:15] Validation started with lhcb-gaussino-new-cmake#136
- [2022-04-12 00:16] Validation started with lhcb-gaussino#1198
- [2022-04-13 00:19] Validation started with lhcb-gaussino-new-cmake#137
- [2022-04-13 00:19] Validation started with lhcb-gaussino-fastsim#254
- [2022-04-13 00:21] Validation started with lhcb-gaussino#1199
Edited by Software for LHCbadded 84 commits
-
3e116ea8...9bcfd500 - 83 commits from branch
master
- b6874fe8 - Merge remote-tracking branch 'origin/master' into MuonMergeGeoCond
-
3e116ea8...9bcfd500 - 83 commits from branch
added ci-test-triggered label
- Resolved by Sebastien Ponce
/ci-test
added hlt2-throughput-decreased label
I've made an attempt to fix the compilation of DD4hep based code with this MR, but I've discovered that this is changing the interface of the DeMuon. Some methods (e.g.
stations
) have completely change meaning and signature. This is not possible as we need to be backward compatible with DetDesc for some time. In particular the algorithms using DeMuon need to be able to call the same methods in DetDesc and DD4hep. This has to be fixed, either by changing DD4hep's interface and mimicing DetDesc or by changing the DetDesc interface and adapting all client algorithms.Now, this being extremely urgent (that is preventing any further progress on DD4hep right now), I propose to modify this MR so that things are compiling even if they are not fully functional. We can then fix remaining issues in a further MR. What do you think ?
- Resolved by Sebastien Ponce
removed lhcb-dd4hep label
mentioned in merge request LHCb!3510 (merged)
mentioned in merge request Rec!2846 (closed)