-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Bugfixes and clean-up after HI mixing updates #7287
Conversation
…added the upper mass cut for low mass analysis
A new Pull Request was created by @yetkinyilmaz for CMSSW_7_4_X. Bugfixes and clean-up after HI mixing updates It involves the following packages: Configuration/Applications @civanch, @nclopezo, @cvuosalo, @boudoul, @franzoni, @mdhildreth, @monttj, @cmsbuild, @srimanob, @slava77, @vadler, @davidlange6 can you please review it and eventually sign? Thanks. |
I did, can you have a look? On 1/26/15 7:58 PM, David Lange wrote:
|
Pull request #7287 was updated. @diguida, @lveldere, @ianna, @cerminar, @covarell, @rcastello, @bendavid, @civanch, @srimanob, @cmsbuild, @ktf, @davidlange6, @vciulli, @Dr15Jones, @cvuosalo, @mdhildreth, @deguio, @slava77, @vadler, @mmusich, @nclopezo, @danduggan, @boudoul, @thuer, @monttj, @franzoni can you please check and sign again. |
naively looks not ok - but I'll check again once we have an IB/release candidate with everything that just changed. |
@yetkinyilmaz - now that I have a current IB available, your rebase is fine. I'm merging as this was signed already |
Bugfixes and clean-up after HI mixing updates
Follow-up on:
#6982
fixing/excluding failed workflows
clean up of obsolete HiMixingModule related options/configurations/products
uniformization of genParticle collection and sequence names