WIP: plan on TDR detector geometry implementation

• Whole detector dimension (preliminary)
  • TDR_o1_v01: TPC + long barrel VXD
  • TDR_o1_v02: TPC + short barrel&endcap VXD
  • TDR_o2_v01: DC + long barrel VXD
  • TDR_o2_v02: DC + short barrel&endcap VXD
• Subsytem
  • MDI
  • VXD
  • Inner silicon tracker
  • Outer silicon tracker & TOF
  • TPC
  • DC
  • Ecal
  • Hcal
  • Coil
  • Yoke & Muon
• Mechanics support & cable

marked the checklist item Ecal as completed

marked the checklist item Ecal as incomplete

marked the checklist item Whole detector dimension (preliminary) as completed

created branch 3-wip-plan-on-tdr-detector-geometry-implementation to address this issue

marked the checklist item VXD as completed

marked the checklist item MDI as completed

marked the checklist item DC as completed

marked the checklist item TPC as completed

marked the checklist item Yoke & Muon as completed

Hi, I may have one minor comment: now there are many print out info from the geometry in several sub-detectors, including: CRDBeamPipe_v01, SiTrackerSkewBarrel_v01, SiTrackerEndcapRing_v01, TPC_HoneyComb_TDR_o1_v01, LongCrystalBarBarrelCalorimeter32Polygon_v01, Lumical_v01. In the release version maybe it's better to mute them.

Best, Fangyi

Yes I fully agree. I am testing dd4hep::printout method to control output level through dd4hep::PrintLevel, such as in Detector/DetCRD/src/Tracker/SiTrackerComposite_v01_geo.cpp and Detector/DetCRD/src/Other/CRDBeamPipe_v01_geo.cpp. If work well, all detector constructor can follow them. Please note that dd4hep default level is dd4hep::INFO, but because dd4hep::DEBUG will print out many dd4hep information, so it had better use dd4hep::INFO as debug, and improve default level to dd4hep::ERROR.

dd4hep::PrintLevel printLevel = dd4hep::ERROR;

if (x_beampipe.hasAttr(_Unicode(printLevel))) {

printLevel = dd4hep::printLevel(x_beampipe.attr<std::string>(_Unicode(printLevel)));

}

dd4hep::PrintLevel oldLevel = dd4hep::setPrintLevel(printLevel);

dd4hep::printout(dd4hep::INFO, "Construct", " ->layer: r0 = %f r1 = %f mat = %s", thickness/dd4hep::mm, thicknessEnd/dd4hep::mm, material.name());

And need recover default level before return.

dd4hep::setPrintLevel(oldLevel);

marked the checklist item Ecal as completed

marked the checklist item Hcal as completed

marked the checklist item Coil as completed

mentioned in merge request !155 (merged)