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
TTStub revision: modernize/update #28435
Conversation
The code-checks are being triggered in jenkins. |
please don't review yet... |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28435/12846
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
@mdhildreth this needs to be rebased... |
@mdhildreth if this is desired for the HLT TDR release, it needs to be ready ASAP; otherwise, it will be delayed till 11_1_X |
@kpedro88 - Yes, I know... I just wanted a placeholder here. I'll work on it this weekend. |
39619ff
to
c0c22d9
Compare
The code-checks are being triggered in jenkins. |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28435/12971
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
The code-checks are being triggered in jenkins. |
Pull request #28435 was updated. @andrius-k, @Dr15Jones, @kmaeshima, @cvuosalo, @schneiml, @benkrikler, @mdhildreth, @ianna, @cmsbuild, @rekovic, @jfernan2, @fioriNTU, @civanch, @kpedro88 can you please check and sign again. |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
+1 |
+1 |
+upgrade |
@mdhildreth could you please make the backport to 11_0_X? I would like to get ready for the final build |
+1 @civanch please check and comment for a further iteration in case |
merge |
PR description:
Updates/modifications/revisions of the TTStub class and the TruthAssociator. Needed for the "modern" version of the L1 Tracking code.
PR validation:
Basic tests completed.