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
Update Scouting Run3 format #36840
Update Scouting Run3 format #36840
Conversation
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36840/28030
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
f718bfa
to
d33cbd6
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36840/28031
|
A new Pull Request was created by @alintulu (Adelina Lintuluoto) for master. It involves the following packages:
@Martin-Grunewald, @Dr15Jones, @smuzaffar, @makortel, @cmsbuild, @missirol can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Hmm, typically we do not add new V2 versions of C++ files but rather modify/evolve the existing code and files. What is the reason here? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here's a first round of comments.
It was decided with @dsperka that keeping the old version would allow us to more easily change producers if necessary. David, maybe you would like to elaborate on this? |
Hmm, I would not want to have variants of code around, based on studies you should settle on one and refine/update that one again if needed (both producer and dataformat), esp. if one is a (almost?) superset of the other. |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36840/28038 ERROR: Build errors found during clang-tidy run.
|
7d804d1
to
752421e
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36840/28148
|
please test |
Pull request #36840 was updated. @makortel, @missirol, @Dr15Jones, @Martin-Grunewald, @smuzaffar can you please check and sign again. |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3cb4b5/22223/summary.html Comparison SummarySummary:
|
+core |
+hlt |
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. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
bestTrack()
has been added to the class Run3ScoutingParticle. The included variables are necessary to run b-tagging with ParticleNet, and has also been identified as helpful for additional analyses groups.doCandIndsForJets
) which dictates if the output should contain the indices of the PF constituents of the jet.PR validation:
I ran
scram b runtests
as well as runTheMatrix. On lxplus runTheMatrix came back with 9 fails but all due toDAS_ERROR
.