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

move to pythia8 219 from tarball #2294

Merged
merged 1 commit into from May 18, 2016

Conversation

mkirsano
Copy link
Contributor

No description provided.

@mkirsano
Copy link
Contributor Author

I suggest this as an intermediate step, taking pythia8 from tarball, tater move to git mirror.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @mkirsano (Mikhail Kirsanov) for branch IB/CMSSW_8_1_X/stable.

@cmsbuild, @smuzaffar, @Degano, @iahmad-khan, @davidlange6 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.

@mkirsano
Copy link
Contributor Author

This is to be tested after merging cms-sw/cmssw#14496 (or together with it)

@bendavid
Copy link
Contributor

@cmsbuild please test with cmssw#14496

@bendavid
Copy link
Contributor

@Degano did I get the syntax wrong or do I just not have permission to trigger tests here?

@ghost
Copy link

ghost commented May 16, 2016

@bendavid I believe the command is correct but has to be issued from cmssw repository. However I'm not entirely sure that's a good idea to have this "intermediate" version as I see no harm in creating a new branch of Pythia in cms-externals and work from there when needed.

@bendavid
Copy link
Contributor

Well there are a number of further changes which depend on this, and no cms patches are needed for now (and hopefully neither in the near future). So it could even stay like this for a while/until a future pythia version. Would be good to avoid additional delays anyways. (We were waiting a while for this release to come from the pythia side)

@cmsbuild
Copy link
Contributor

-1

Tested at: 83b0ff7

  • Build:

I found an error when building:

Package cms+coral+CORAL_2_3_21-cms2 not found in repository. Queuing for build.
Dependencies for cms+coral+CORAL_2_3_21-cms2: ['install-external+gcc+5.3.0-cms', 'install-lcg+SCRAMV1+V2_2_6_pre5-cms', 'install-cms+coral-tool-conf+2.1-cms2', 'install-external+dwz+0.11-cms', 'install-external+file+5.13-cms', 'fetch-cms+coral+CORAL_2_3_21-cms2']
Starting to process package external+hector+1.3.4_patch1-cms2
Checking if hector is cached.
Package external+hector+1.3.4_patch1-cms2 not found in repository. Queuing for build.
Error while downloading ftp://ftp.cwru.edu/pub/bash/readline-6.2.tar.gz: 
Dependencies for external+hector+1.3.4_patch1-cms2: ['install-external+gcc+5.3.0-cms', 'install-lcg+root+6.06.04', 'fetch-external+hector+1.3.4_patch1-cms2']
Starting to process package external+pyminuit2+0.0.1-cms2
Checking if pyminuit2 is cached.
Package external+pyminuit2+0.0.1-cms2 not found in repository. Queuing for build.
Dependencies for external+pyminuit2+0.0.1-cms2: ['install-external+gcc+5.3.0-cms', 'install-lcg+root+6.06.04', 'fetch-external+pyminuit2+0.0.1-cms2']


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

@bendavid
Copy link
Contributor

This failure looks rather unrelated to the pythia update?

@smuzaffar
Copy link
Contributor

correct, looks like it was using the wrong arch slc6_amd64_gcc530 for IB/CMSSW_8_1_X/stable branch. @Degano can you please look it to it.
for now I have re-started the test using the correct arch (slc6_amd64_gcc493)

@cmsbuild
Copy link
Contributor

-1

Tested at: 83b0ff7

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

I found follow errors while testing this PR

Failed tests: UnitTests

  • Unit Tests:

I found errors in the following unit tests:

---> test testRecoMETMETProducers had ERRORS

@bendavid
Copy link
Contributor

Also seems unlikely this failure is related (and the linked summary report seems not to be there)

@cmsbuild
Copy link
Contributor

cmsbuild commented May 18, 2016

The tests are being triggered in jenkins.
Using cmssw from cms-sw/cmssw#14496
https://cmssdt.cern.ch/jenkins/job/cmsdist-cmssw-test-pr/46/console

@davidlange6 davidlange6 merged commit c9af3fa into cms-sw:IB/CMSSW_8_1_X/stable May 18, 2016
@cmsbuild
Copy link
Contributor

-1

Tested at: 83b0ff7

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

I found follow errors while testing this PR

Failed tests: Build

  • Build:

I found an error when building:

>> Compiling  /build/cmsbuild/jenkins-workarea/workspace/cmsdist-cmssw-test-pr/CMSSW_8_1_X_2016-05-17-1100/src/JetMETCorrections/MCJet/bin/ResponseFitter.cc 
>> Compiling  /build/cmsbuild/jenkins-workarea/workspace/cmsdist-cmssw-test-pr/CMSSW_8_1_X_2016-05-17-1100/src/JetMETCorrections/MCJet/bin/L2Correction.cc 
>> Compiling  /build/cmsbuild/jenkins-workarea/workspace/cmsdist-cmssw-test-pr/CMSSW_8_1_X_2016-05-17-1100/src/JetMETCorrections/MCJet/bin/L3Correction.cc 
>> Compiling  /build/cmsbuild/jenkins-workarea/workspace/cmsdist-cmssw-test-pr/CMSSW_8_1_X_2016-05-17-1100/src/JetMETCorrections/MCJet/bin/ReadTree.cc 
/build/cmsbuild/jenkins-workarea/workspace/cmsdist-cmssw-test-pr/CMSSW_8_1_X_2016-05-17-1100/src/Utilities/StaticAnalyzers/src/FiniteMathChecker.cc: In member function 'void clangcms::FiniteMathChecker::checkPreStmt(const clang::CallExpr*, clang::ento::CheckerContext&) const':
/build/cmsbuild/jenkins-workarea/workspace/cmsdist-cmssw-test-pr/CMSSW_8_1_X_2016-05-17-1100/src/Utilities/StaticAnalyzers/src/FiniteMathChecker.cc:33:51: error: no matching function for call to 'clang::ento::CheckerContext::generateSink()'
   clang::ento::ExplodedNode *N = ctx.generateSink();
                                                   ^
/build/cmsbuild/jenkins-workarea/workspace/cmsdist-cmssw-test-pr/CMSSW_8_1_X_2016-05-17-1100/src/Utilities/StaticAnalyzers/src/FiniteMathChecker.cc:33:51: note: candidate is:
In file included from /build/cmsbuild/jenkins-workarea/workspace/cmsdist-cmssw-test-pr/CMSSW_8_1_X_2016-05-17-1100/src/Utilities/StaticAnalyzers/src/FiniteMathChecker.h:5:0,
                 from /build/cmsbuild/jenkins-workarea/workspace/cmsdist-cmssw-test-pr/CMSSW_8_1_X_2016-05-17-1100/src/Utilities/StaticAnalyzers/src/FiniteMathChecker.cc:1:


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