New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding a Phase2 Tracker (T11) and corresponding workflow (D29) for testing and relvals purpose #23720
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-23720/5375 |
A new Pull Request was created by @boudoul (boudoul) for master. It involves the following packages: Configuration/Geometry @prebello, @Dr15Jones, @ianna, @mdhildreth, @fabozzi, @cmsbuild, @franzoni, @kpedro88, @civanch, @GurpreetSinghChahal, @fabiocos, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
@boudoul we can't launch the tests with the new workflow unless it's added to relval_2023 (it's fine with me to skip testing it explicitly here, we didn't test all the other tracker variations) |
Comparison is ready There are some workflows for which there are errors in the baseline: Comparison Summary:
|
+1 |
+1 |
sorry approved by mistake.. I am checking it |
@kpedro88 not clear to me: are you going to skip phaseII workflows (D29 and D30) test as pointed above? |
yes, we'll skip it for now |
+1 |
@fabiocos can this go in? |
+operations addition of a new upgrade geometry scenario |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 workflow 24407.0 runs smoothly |
Greetings
This PR aims to add a new version of the phase2 tracker for testing purpose as mentioned at the last simulation meeting with @kpedro88
This new Tracker (T11) is the same as T6 but 50x50 modules.
This Tracker would benefit to get relvals for physics performance in order to guide R&Ds choice and complete the picture (together with T6,T7,T8)
The corresponding Workflows for T11 is D29 (244xx)
I didn't add it in IB test, I don't think it's needed , but it can be added if prefered.
The PR test should therefore be launched with a 244xx WF
@emiglior FYI