-
Notifications
You must be signed in to change notification settings - Fork 5
Add JSON for LST-seeded iteration (phase-2 HLT) #19
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
Conversation
|
A new Pull Request was created by @mmasciov for branch main. @akritkbehera, @cmsbuild, @iarspider, @jfernan2, @mandrenguyen, @smuzaffar, @srimanob can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
test parameters:
|
|
@cmsbuild, please test |
@mmasciov for my own education is the additional json file forseen to be used for offline tracking? |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-18dd46/49447/summary.html Comparison SummarySummary:
|
That's correct: it's just meant to facilitate mkFit developers in the context of future offline-oriented developments. |
|
please test (for completeness, as cms-sw/cmssw#49383 was updated) |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-18dd46/49459/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: You can see more details here: Comparison SummarySummary:
|
thanks for the clarification. |
|
REMINDER @sextonkennedy, @mandrenguyen, @ftenchini: This PR was tested with cms-sw/cmssw#49383, please check if they should be merged together |
|
@jfernan2 @iarspider can you sign this one too given that cms-sw/cmssw#49383 has converged? |
|
merge |
|
+1 |
As per title.
In addition, a JSON for highPtTripletStep in phase-2 is added, in view of further, future developments.