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
Revert new cpe #5581
Revert new cpe #5581
Conversation
…NewCPE Conflicts: RecoLocalTracker/SiStripRecHitConverter/python/StripCPEfromTrackAngle_cfi.py
A new Pull Request was created by @nickmccoll for CMSSW_7_2_X. Revert new cpe It involves the following packages: RecoLocalTracker/SiStripRecHitConverter @cmsbuild, @nclopezo, @StoyanStoynev, @slava77 can you please review it and eventually sign? Thanks. |
This PR fixes the issues described here: https://hypernews.cern.ch/HyperNews/CMS/get/relval/3159/18/1.html |
@nickmccoll please provide a proper description to the PR. |
Hi @StoyanStoynev, which other information do you need? This PR just reverts one parameter that is responsible for the differences spotted in validation of pre6 (see link above) |
@cerati You are not saying the current description which is "No description provided." is the proper one, are you? We need these for documentation purposes (at least). One can refer to the PR that is reverted and give link to the reasoning (the one in the comment). It is good to actually summarize the reasoning and to say what the expected effect from the PR is. |
I guess only Nick has access to the description field, if this is what is needed. This PR reverts 5151 (keeping the code in release, it will be used later on, after more tests/developments are done). Please start the review process. |
I am trying to merge this PR with the latest IB, CMSSW_7_2_X_2014-09-26-0800, and I get |
On 26 Sep, 2014, at 4:55 PM, Slava Krutelyov notifications@github.com wrote:
|
@Martin-Grunewald |
I guess I learned my lesson: better follow the policy of no tracking changes in the last open pre. |
+1 tested in CMSSW_7_2_X_2014-09-24-1400 (test area test5581; it was more convenient to test in an older area) This seems like a case of Le mieux est l'ennemi du bien. What are we going to do in 73X? |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes (tests are also fine). @nclopezo can you please take care of it? |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes (tests are also fine). @nclopezo can you please take care of it? |
This PR fixes the issues described here: https://hypernews.cern.ch/HyperNews/CMS/get/relval/3159/18/1.html
It changes the default strip CPE to the original version due to an unexpected drop in tracking efficiency.