Skip to content

Preliminary work to support DD4HEP in RICH Reco.

Christopher Rob Jones requested to merge jonrob/LHCb:TestRichDD4HEP into master

This MR adds preliminary support for using DD4HEP classes with the RICH reconstruction.

Support is not yet complete, as there are currently a number of missing bits and pieces of functionality in DD4HEP which will need to be addressed. The approach I have taken in this MR is, when building for dd4hep to use the dd4hep classes wherever possible, but where support is missing to defer back to the DetDesc classes. The plan then, going forward is to work to address and ultimately remove all the places where DetDesc still needs to be used.

Note this MR requires Detector!143 (merged) lhcb-conddb/lhcb-conditions-database!8 (merged) Lbcom!595 (merged) Rec!2692 (merged)

This MR also include the commits in !3380 (merged) as they are required to support the RICH.

@seaso @bcouturi FYI

Note:-

Changes to reference logs are expected, as some of the changes to how conditions data is cached is not bit-wise identical to before, and this causes some noise in counters etc. diffs though should not be significant.

Larger diffs though are expect in the dd4hep builds, as the geometry information is not identical (e.g. rich2 mirror parameters) to the DetDesc geometry, which means larger diffs are expected in these builds.

Edited by Christopher Rob Jones

Merge request reports