Survey for UT modules and stave faces, correction to noise condition etc

mentioned in merge request SIMCOND!143 (merged)

This is replacing the dummy values (e.g. zeros for the positions) with survey measurements from when the UT was closed. The condition is modified instead of creating a new IOV range but I think this is fine in this case.

I don't think this needs testing as the UT reconstruction tests are disabled in the DD4hep builds.

@mfontana @lgrillo @freiss @jonrob @clemenci what do you think?

@hawu, will this work with UT as in Detector/HEAD and lhcb/Detector!395 (merged) too?

Yes, this branch should have the same structure as in current master but only with values updated.

However, I actually not sure if it is legit to have these realistic Alignment values go into the 0 files. If they need to be tagged properly, please let me know.

@hawu, as far as I understood the 0 files contain the default. If these are your first values that you put in, to me it is reasonable to consider the first closed survey as the default.

One more comment, for me it makes sense also to use the survey closed position in the simulation so we should propagate it to the Sim10/2023 branch for future simulations with DD4Hep.

While we can keep a perfect detector for Sim10/Run3.

added cherry-pick to simulation label

approved this merge request

assigned to @clemenci

changed the description

merged

mentioned in commit 433a3401

resolved all threads

mentioned in merge request lhcb/Detector!395 (merged)