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
switch to Pandora recipe with no fragment removal algorithms #8856
Conversation
A new Pull Request was created by @kpedro88 (Kevin Pedro) for CMSSW_6_2_X_SLHC. switch to Pandora recipe with no fragment removal algorithms It involves the following packages: RecoParticleFlow/PandoraTranslator The following packages do not have a category, yet: RecoParticleFlow/PandoraTranslator @cmsbuild, @nclopezo can you please review it and eventually sign? Thanks. |
Do you want this as the default for all pileup scenarios? We could put something in the customisation that checks what pileup it is and decide which config to run. |
I'm not sure what would be best at this moment. @lgray, @vandreev11, what do you think? |
Busy right now, back to you later tonight. (Sent from my Nexus 6)
|
On my opinion it should be the same for all PU scenarios. PU0 could be a special case, but it is better to avoid that for the sake of comparisons, drawing conclusions. I believe it is temporary measure, clustering in Pandora needs to be tuned yet for high pileup case. |
Along the same lines as Valeri, since we're going to be doing a massive We'll likely still be using some of the linear collider algorithms even -L (Sent from my Nexus 6)
|
merge Okay, so that's everything for the next patch release right? Getting on a plane now, I'll put in the build request when I land. |
switch to Pandora recipe with no fragment removal algorithms
Yes, that should be everything. |
This commit adds a modified Pandora recipe with no fragment removal algorithms, and sets it as the default recipe in the python configuration. This increases the speed of Pandora in high-pileup environments with little effect on the jet resolution, see attached slides:
attn: @lgray, @vandreev11, @pfs, @boudoul