Small fixes for DD4hep support of alignment

code cleanup, ability to explicitely drop slices and workaround for a strange dependency issue in makefiles

Goes along with Detector!84 (merged), Rec!2396 (merged), Alignment!168 (merged) and AlignmentOnline!49 (closed).

added lhcb-dd4hep label

added 2 commits

• afd29c51 - Small fixes for DD4hep support of alignment
• cc644801 - Workaround dependency issues in DD4hep

Compare with previous version

mentioned in merge request Detector!84 (merged)

changed the description

mentioned in merge request Rec!2396 (merged)

mentioned in merge request Alignment!168 (merged)

assigned to @msaur

@sponce In description related MRs Detector!84 (merged), Rec!2396 (merged) and Alignment!168 (merged) are mentioned, but Alignment!168 (merged) also depends on AlignmentOnline!49 (closed), I presume that one must be included in test as well, right?

Right, I forgot to duplicate that dependency in all MRs. Sorry

You know what, I'll launch the ci-test myself when this is ready as I still need to sort out the piepeline plus I want to recompile everything locally without DD4hep, but I needed the MRS to make that easy.

changed the description

added 38 commits

• cc644801...7c054c85 - 36 commits from branch master
• cc567bb6 - Small fixes for DD4hep support of alignment
• 60dfd7f3 - Workaround dependency issues in DD4hep

Compare with previous version

• [2021-04-09 17:01] Validation started with lhcb-master-mr#2168
• [2021-04-10 00:06] Validation started with lhcb-dd4hep#797
• [2021-04-12 10:55] Validation started with lhcb-master-mr#2175
• [2021-04-13 00:04] Validation started with lhcb-dd4hep#798
• [2021-04-13 12:06] Validation started with lhcb-master-mr#2188
• [2021-04-14 00:06] Validation started with lhcb-dd4hep#799
• [2021-04-15 00:05] Validation started with lhcb-dd4hep#800
• [2021-04-16 00:08] Validation started with lhcb-dd4hep#801
• [2021-04-16 01:28] Validation started with lhcb-dd4hep#801

mentioned in issue Moore#265 (closed)

assigned to @jbaptist and unassigned @msaur

Hi @sponce, please let me know when it can be tested. Thanks

Hi @jbaptist, this is a quite complicated case as it involves 5 MRs in 5 different projects and 2 slots (needs testing both in regular mode with ci-test and in DD4hep mode in the dedicated slot). On top I expect several rounds before it stabilizes. So I'll basically handle the testing myself (see Alignment!168 (comment 4381787) for example) and let you know when all is fine.

added 3 commits

• afc09030 - Small fixes for DD4hep support of alignment
• 774af8cb - Workaround dependency issues in DD4hep
• 609d0879 - Revert "Do not trigger updateManagerSvc when setting DeltaMatirx"

Compare with previous version