Skip to content
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

update to gensim recycling that matches beamspot in GT #12198

Merged
merged 1 commit into from Oct 30, 2015

Conversation

davidlange6
Copy link
Contributor

at least for most samples. More inputs are needed before this issue is completely fixed.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @davidlange6 (David Lange) for CMSSW_7_6_X.

update to gensim recycling that matches beamspot in GT

It involves the following packages:

Configuration/PyReleaseValidation

@danduggan, @boudoul, @franzoni, @fabozzi, @cmsbuild, @srimanob can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald 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.
If you are a L2 or a release manager you can ask for tests by saying 'please test' or '@cmsbuild, please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@davidlange6
Copy link
Contributor Author

@cmsbuild, please test

@cmsbuild
Copy link
Contributor

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/9359/console

@cmsbuild
Copy link
Contributor

davidlange6 added a commit that referenced this pull request Oct 30, 2015
update to gensim recycling that matches beamspot in GT
@davidlange6 davidlange6 merged commit 59c27a4 into cms-sw:CMSSW_7_6_X Oct 30, 2015
@cmsbuild
Copy link
Contributor

@davidlt
Copy link
Contributor

davidlt commented Nov 1, 2015

This is causing 1350.0 and 1349.0 to fail for now.

@hengne
Copy link
Contributor

hengne commented Nov 1, 2015

@davidlt @davidlange6
The reason is the GENSIMs for the two workflows are failed:
1.) 1350.0_JpsiMuMu_Pt-15 is failed because of the outdated gen module (which is still using pythia6) is removed from the release.
2.) 1349.0_BsToMuMu_13 fail due to running too long beause of the very low generator level efficiency.

According to BPH, they still need these two workflows, and seeking for solution with generator team.

So, a temporary solution is to comment out the "xxxINPUT" lines for the two workflows. Shall we do this? I can sent a PR to fix this if we are agree on this solution. Oh, for 1350.0_JpsiMuMu_Pt-15 we should actually comment out the whole workflow, because the gen step would also crash due to the missing (removed because of out date) gen module.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants