change weight file names

[dbenedet]

New weight files for the trackerDriven seeding need to be uploaded.
They are 9 xml files. Could you also take care of the request in cmsdist?

The branch with all the code changes will come tomorrow.

[cmsbuild]

A new Pull Request was created by @dbenedet for CMSSW_7_1_X.

change weight file names

It involves the following packages:

RecoParticleFlow/PFTracking

@nclopezo, @cmsbuild, @anton-a, @thspeer, @slava77, @Degano can you please review it and eventually sign? Thanks.
@bachtis, @lgray this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@nclopezo, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[slava77]

Hi Daniele,

What's the effect of these new weight files?

The download.url is not really used anymore.
So, this PR is not needed for the release.
It is, however useful to know that there is an update.

Giulio, how should we handle these? (is attention of the release managers enough in this case)

[dbenedet]

Is updating the trackerDriven seeding after 5 years :) ... the effect are reported on Sachiko talk. A bit more efficiency for the same fake rate:
https://indico.cern.ch/event/305108/contribution/1/material/slides/0.pdf

I tested in pre6 and I confirmed the results. Unfortunately there were many changes on GoodSeedProducer for pre7 so when rebasing I needed to solve quite many conflict. I am re-checking the results now.

[slava77]

Hi Daniele,

Thank you for the summary of the effects.
It helps a lot to document the release changes with some details in one place.

[cmsbuild]

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3678/1292/summary.html

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3678/1292/summary.html

[slava77]

+1

this PR is pointless though. The download.url files are not used anymore

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_1_X IBs unless changes (tests are also fine). @nclopezo, @ktf can you please take care of it?

[ktf]

Indeed. @Degano did you take care of the external package?