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
RecoParticleFlow/PFProducer - some cleanup and modernization #26519
Conversation
The code-checks are being triggered in jenkins. |
Hi @slava77, I just saw you commented in my last PR which was closed (so I didn't get an email notification). Right, the includes! I should quickly clean them up. Moving the algo in the plugins directory would be a bit too much I think, because that would apply to all the algos actually. And moving only one would be a bit inconsistent... |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-26519/9381
|
A new Pull Request was created by @guitargeek (Jonas Rembser) for master. It involves the following packages: RecoParticleFlow/PFProducer @perrotta, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
The code-checks are being triggered in jenkins. |
@cmsbuild please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-26519/9382
|
The tests are being triggered in jenkins. |
Comparison job queued. |
The code-checks are being triggered in jenkins. |
|
||
|
||
|
||
private : | ||
|
||
/// Analyse nuclear interactions where a primary or merged track is present | ||
void analyseNuclearWPrim(std::unique_ptr<reco::PFCandidateCollection>&, unsigned int); | ||
void analyseNuclearWPrim(reco::PFCandidateCollection&, std::vector<bool> &, unsigned int) const; | ||
|
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.
I updated with force-push because my mistake that caused the differences was not interesting enough for it's own commit: I forgot to pass the mask vector by reference (also in the other function)...
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-26519/9387
|
@cmsbuild please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready There are some workflows for which there are errors in the baseline: Comparison Summary:
|
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 |
PR description:
After my last slightly unpopular PR with the PFBlockAlgo I'm back with some other quick ideas on simplifying and modernizing the PF code :)
unique_ptr
to sub-algos instead of raw pointersconst
member functions and avoids smart pointersunique_ptr
in the PFCandConnectordist
member is actually used => the class can just be replaced with adouble
Furthermore, the includes in the PFBlockAlgo/PFBlockProducer were cleaned up. For this, I used the necessary includes found by the include-what-you-use (iwyu) tool in my old PR and spread them across the 4 files as appropriate.
PR validation:
Code compiles and matrix tests complete.