Draft: NOT TO MERGE Fix FT in DD4HEP

Goes with Rec!3065 (closed)

• It seems the stereoAngle of the layers is not propagated well: in the geometry they seem to be considered vertical.
• The calculation of size using Boxes does not work
• Moved the p.leftHoleSizeY / p..rightHoleSizeY out of the 0.5 factor, for no other reason than "it seems to work"

assigned to @lohenry

mentioned in merge request Rec!3065 (closed)

mentioned in issue Rec#382

changed the description

added dd4hep lhcb-dd4hep not for merge labels

• [2022-08-05 00:03] Validation started with lhcb-dd4hep-rootunits#9
• [2022-08-05 00:03] Validation started with lhcb-dd4hep#1222
• [2022-08-06 00:03] Validation started with lhcb-dd4hep-rootunits#10
• [2022-08-06 00:03] Validation started with lhcb-dd4hep#1223
• [2022-08-10 00:04] Validation started with lhcb-dd4hep-rootunits#11
• [2022-08-10 00:04] Validation started with lhcb-dd4hep#1224
• [2022-08-11 00:03] Validation started with lhcb-dd4hep-rootunits#12
• [2022-08-11 00:04] Validation started with lhcb-dd4hep#1225
• [2022-08-12 00:03] Validation started with lhcb-dd4hep#1226
• [2022-08-12 00:05] Validation started with lhcb-dd4hep-rootunits#13
• [2022-08-13 00:03] Validation started with lhcb-dd4hep#1227
• [2022-08-13 00:03] Validation started with lhcb-dd4hep-rootunits#14
• [2022-08-13 13:53] Validation started with lhcb-dd4hep#1228
• [2022-08-14 00:03] Validation started with lhcb-dd4hep-rootunits#15
• [2022-08-14 00:03] Validation started with lhcb-dd4hep#1229
• [2022-08-14 19:20] Validation started with lhcb-dd4hep#1230
• [2022-08-15 00:05] Validation started with lhcb-dd4hep-rootunits#16
• [2022-08-15 00:05] Validation started with lhcb-dd4hep#1231
• [2022-08-16 00:04] Validation started with lhcb-dd4hep#1232
• [2022-08-16 00:04] Validation started with lhcb-dd4hep-rootunits#17
• [2022-08-16 14:21] Validation started with lhcb-dd4hep#1233
• [2022-08-17 00:03] Validation started with lhcb-dd4hep#1234
• [2022-08-17 00:03] Validation started with lhcb-dd4hep-rootunits#18
• [2022-08-18 00:03] Validation started with lhcb-dd4hep-rootunits#19
• [2022-08-18 00:04] Validation started with lhcb-dd4hep#1235
• [2022-08-18 11:21] Validation started with lhcb-dd4hep#1236
• [2022-08-18 16:30] Validation started with lhcb-dd4hep#1237
• [2022-08-18 18:00] Validation started with lhcb-dd4hep#1238
• [2022-08-19 00:03] Validation started with lhcb-dd4hep-rootunits#20
• [2022-08-19 00:03] Validation started with lhcb-dd4hep#1239
• [2022-08-20 00:03] Validation started with lhcb-dd4hep#1240
• [2022-08-21 00:03] Validation started with lhcb-dd4hep#1241

• Moved the p.leftHoleSizeY / p..rightHoleSizeY out of the 0.5 factor, for no other reason than "it seems to work"

Concerning this: this makes the beam pipe hole too big. The actual cause of this problem in the y positions of the mirror points being wrong I traced back to the fibre box dimensions: the fibre lengths are still set to 2424mm even for the inner most modules.

mentioned in merge request !271 (merged)

removed lhcb-dd4hep label

closed