Survey for UT modules and stave faces, correction to noise condition etc
Merge request reports
Activity
mentioned in merge request SIMCOND!143 (merged)
- Resolved by Marco Clemencic
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?
- Resolved by Marco Clemencic
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.Edited by Hangyi Wu
@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.added cherry-pick to simulation label
assigned to @clemenci
mentioned in commit 433a3401
mentioned in merge request lhcb/Detector!395 (merged)