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

upgrade to lhapdf 6.2.1 #3415

Merged
merged 1 commit into from Sep 15, 2017
Merged

Conversation

mkirsano
Copy link
Contributor

No description provided.

@cmsbuild
Copy link
Contributor

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

@cmsbuild, @smuzaffar, @mrodozov 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.

@perrozzi
Copy link

@cmsbuild please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 13, 2017

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

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

Comparison job queued.

@cmsbuild
Copy link
Contributor

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3415/22959/summary.html

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 1220 differences found in the comparisons
  • DQMHistoTests: Total files compared: 26
  • DQMHistoTests: Total histograms compared: 2646524
  • DQMHistoTests: Total failures: 11406
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 2634929
  • DQMHistoTests: Total skipped: 189
  • DQMHistoTests: Total Missing objects: 0
  • Checked 107 log files, 14 edm output root files, 26 DQM output files

@perrozzi
Copy link

perrozzi commented Sep 14, 2017

dear all, these (#3415, #3416, #3417, #3419) are the urgent updates needed by GEN for MC v2 production we were discussing at the last ORP.
Please consider approving them at your earliest convenience.
Thanks a lot in advance

This was referenced Sep 14, 2017
@smuzaffar
Copy link
Contributor

+external

@smuzaffar smuzaffar merged commit 469916f into cms-sw:IB/CMSSW_9_4_X/gcc630 Sep 15, 2017
@perrozzi
Copy link

Just to clarify: the backport of this PR in particular to 92x (#3419) and 71x (#3417) and corresponding new releases are very urgent because we need the new PDF to start producing gridpacks [*]. The gridpacks production is a pre-condition for the GEN-SIM campaign, it would be good not to wait for 930 to be out.

[*] We use 92x (gcc53x) for POWHEG and 71x (gcc48x) for Madgraph gridpacks, they can be then used on all the CMSSW versions

@davidlange6
Copy link
Contributor

davidlange6 commented Sep 15, 2017 via email

@perrozzi
Copy link

Yes I agree, but for the moment we have validated POWHEG using gcc530. We were not aware of the switch to gcc630 for MC v2 production.
We can try to validate it asap in 93x/gcc630, hoping that there are no hiccups. I assume that it would be ok.
Anticipating possible problems, what shall we do in case gcc630 doesn't work?

@davidlange6
Copy link
Contributor

davidlange6 commented Sep 15, 2017 via email

@perrozzi
Copy link

I am collecting the issues here
cms-sw/genproductions#1307 (comment)
they are not necesarily related (only) to gcc630...
For the moment we can probably live with 93x, I will let you know in case of "unsolvable" problems

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