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

Lhe interface update for pythia 8.205 (74x) #7447

Merged
merged 2 commits into from Jan 29, 2015

Conversation

bendavid
Copy link
Contributor

Must be taken together with pythia update in cmsdist
cms-sw/cmsdist#1340

…is no longer needed in pythia 8.205 when multiplicities are steered automatically
@bendavid
Copy link
Contributor Author

+1

@cmsbuild cmsbuild added this to the Next CMSSW_7_4_X milestone Jan 29, 2015
@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes or unless it breaks tests.

@cmsbuild
Copy link
Contributor

-1
Tested at: ab1aab3
I found an error when building:

>> Package FWCore/Version built
>> Subsystem FWCore built
>> Building edm plugin tmp/slc6_amd64_gcc491/src/GeneratorInterface/Pythia8Interface/test/ZJetsTestAnalyzer/libZJetsTestAnalyzer.so
>> Building shared library tmp/slc6_amd64_gcc491/src/GeneratorInterface/Pythia8Interface/src/GeneratorInterfacePythia8Interface/libGeneratorInterfacePythia8Interface.so
/build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_7_4_X_2015-01-28-1400/src/GeneratorInterface/Pythia8Interface/plugins/Pythia8Hadronizer.cc: In member function 'bool Pythia8Hadronizer::initializeForExternalPartons()':
/build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_7_4_X_2015-01-28-1400/src/GeneratorInterface/Pythia8Interface/plugins/Pythia8Hadronizer.cc:442:37: error: invalid new-expression of abstract class type 'LHAupLesHouches'
     lhaUP.reset(new LHAupLesHouches());
                                     ^
In file included from /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_7_4_X_2015-01-28-1400/src/GeneratorInterface/Pythia8Interface/plugins/Pythia8Hadronizer.cc:43:0:
/build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_7_4_X_2015-01-28-1400/src/GeneratorInterface/Pythia8Interface/plugins/LHAupLesHouches.h:22:7: note:   because the following virtual functions are pure within 'LHAupLesHouches':
 class LHAupLesHouches : public Pythia8::LHAup {


you can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7447/2217/summary.html

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (but tests are reportedly failing).

davidlange6 added a commit that referenced this pull request Jan 29, 2015
Lhe interface update for pythia 8.205 (74x)
@davidlange6 davidlange6 merged commit 90f23a0 into cms-sw:CMSSW_7_4_X Jan 29, 2015
@davidlange6
Copy link
Contributor

failures expected without cmsdist changes. I have merged both

@Martin-Grunewald
Copy link
Contributor

This breaks jenkins tests! Please fix or revert!

bendavid pushed a commit to bendavid/cmssw that referenced this pull request Jan 30, 2015
Lhe interface update for pythia 8.205 (74x)
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