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

Topythia8183 62 x #2829

Merged
9 commits merged into from
Mar 21, 2014
Merged

Topythia8183 62 x #2829

9 commits merged into from
Mar 21, 2014

Conversation

mkirsano
Copy link
Contributor

No description provided.

@mkirsano
Copy link
Contributor Author

To be made together with the corresponding change of cmsdist

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @mkirsano for CMSSW_6_2_X.

Topythia8183 62 x

It involves the following packages:

GeneratorInterface/Pythia8Interface

@vciulli, @nclopezo, @thuer, @cmsbuild, @bendavid, @Degano can you please review it and eventually sign? Thanks.
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.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@ghost
Copy link

ghost commented Mar 13, 2014

+1
Tested locally alongside with cmsdist PR: cms-sw/cmsdist#497 with the usual runTheMatrix workflows, that is 401.0,1306.0,101.0,1003.0,50101.0.
All the tests passed.

@mkirsano
Copy link
Contributor Author

I added this commit to the same pull request on request of Josh, he hopes that 6_2_X will converge faster like this.

@mkirsano
Copy link
Contributor Author

This is a backport of the slha code from 7_1_X

@cmsbuild
Copy link
Contributor

Pull request #2829 was updated. @vciulli, @nclopezo, @thuer, @cmsbuild, @bendavid, @Degano can you please check and sign again.

@bendavid
Copy link
Contributor

I will actually run this through some tests with lhe files from amc@nlo, which should also explicitly test the slha functionality.

@bendavid
Copy link
Contributor

-1
I think you missed the include changes in LHEInterface (dependent package which you get with git-cms-checkdeps -a for example)

/afs/cern.ch/work/b/bendavid/CMSSWgen2/CMSSW_6_2_7_patch2/src/GeneratorInterface/LHEInterface/plugins/Pythia8Hadronisation.cc:14:20: fatal error: Pythia.h: No such file or directory
compilation terminated.
gmake: *** [tmp/slc5_amd64_gcc472/src/GeneratorInterface/LHEInterface/plugins/GeneratorInterfacePythia8Hadronisation/Pythia8Hadronisation.o] Error 1
gmake: *** Waiting for unfinished jobs....
gmake: *** [There are compilation/build errors. Please see the detail log above.] Error 2

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_6_2_X IBs unless changes or unless it breaks tests. @Degano can you please take care of it?

@cmsbuild
Copy link
Contributor

-1
I found an error when building:

>> Compiling  /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/LHEInterface/src/JetMatching.cc 
>> Compiling  /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/LHEInterface/src/LHEEvent.cc 
>> Compiling  /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/LHEInterface/src/JetClustering.cc 
>> Compiling  /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/LHEInterface/src/JetMatchingMLM.cc 
In file included from /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/Pythia8Interface/src/RandomP8.cc:1:0:
/build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/Pythia8Interface/interface/RandomP8.h:1:28: fatal error: Pythia8/Pythia.h: No such file or directory
compilation terminated.
In file included from /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/Pythia8Interface/src/RandomP8.cc:1:0:
/build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/Pythia8Interface/interface/RandomP8.h:1:28: fatal error: Pythia8/Pythia.h: No such file or directory
compilation terminated.
gmake: **\* [tmp/slc6_amd64_gcc472/src/GeneratorInterface/Pythia8Interface/src/GeneratorInterfacePythia8Interface/RandomP8.o] Error 1


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

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_6_2_X IBs unless changes (but tests are reportedly failing). @Degano can you please take care of it?

@mkirsano
Copy link
Contributor Author

Hello, this happens if it is not synchronized with the change of
pythia8 version in cmsdist.

On 03/19/2014 11:11 AM, cmsbuild wrote:

-1
I found an error when building:

Compiling /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/LHEInterface/src/JetMatching.cc

Compiling /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/LHEInterface/src/LHEEvent.cc

Compiling /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/LHEInterface/src/JetClustering.cc

Compiling /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/LHEInterface/src/JetMatchingMLM.cc

In file included from /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/Pythia8Interface/src/RandomP8.cc:1:0:

/build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/Pythia8Interface/interface/RandomP8.h:1:28: fatal error: Pythia8/Pythia.h: No such file or directory

compilation terminated.

In file included from /build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/Pythia8Interface/src/RandomP8.cc:1:0:

/build/cmsbuild/jenkins-workarea/workspace/ib-integration-CMSSW_6_2_X-slc6_amd64_gcc472/CMSSW_6_2_X_2014-03-19-0200/src/GeneratorInterface/Pythia8Interface/interface/RandomP8.h:1:28: fatal error: Pythia8/Pythia.h: No such file or directory

compilation terminated.

gmake: *** [tmp/slc6_amd64_gcc472/src/GeneratorInterface/Pythia8Interface/src/GeneratorInterfacePythia8Interface/RandomP8.o] Error 1

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


Reply to this email directly or view it on GitHub
#2829 (comment).

@ghost
Copy link

ghost commented Mar 19, 2014

Yes, sorry, I am re-running the tests with external PR right now.
Could you also please make a PR for Pythia 8.183 in cmsdist for the branch 6_2_X/devel_gcc472 which is the branch used for slc6 that does not merges from 6_2_X/stable?

@mkirsano
Copy link
Contributor Author

OK, but the priority now is 5_3_X

On 03/19/2014 11:33 AM, Alessandro Degano wrote:

Yes, sorry, I am re-running the tests with external PR right now.
Could you also please make a PR for Pythia 8.183 in cmsdist for the
branch 6_2_X/devel_gcc472 which is the branch used for slc6 that does
not merges from 6_2_X/stable?


Reply to this email directly or view it on GitHub
#2829 (comment).

@ghost
Copy link

ghost commented Mar 19, 2014

+1
Tested locally.

@cmsbuild
Copy link
Contributor

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

@vciulli
Copy link
Contributor

vciulli commented Mar 19, 2014

@mkirsano: the new 6_2_X is going to be built next week, while 5_3_X is the week after so please try to go ahead first with the backporting in 6_2_X (this is equally important than the one to 5_3_X)

@ghost
Copy link

ghost commented Mar 21, 2014

I tested it also against the slc6 cmsdist PR cms-sw/cmsdist#508 and everything works fine there too.

@davidlange6
Copy link
Contributor

+1

ghost pushed a commit that referenced this pull request Mar 21, 2014
Generator fixes -- Topythia8183 62 x
@ghost ghost merged commit ca5158b into cms-sw:CMSSW_6_2_X Mar 21, 2014
@ghost ghost mentioned this pull request Mar 24, 2014
This pull request was closed.
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

5 participants