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

adding VBFNLO and Hjets++ to the externals linked to Herwig7 #4753

Merged
merged 5 commits into from May 21, 2019

Conversation

Andrej-CMS
Copy link
Contributor

No description provided.

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 6, 2019

A new Pull Request was created by @Andrej-CMS for branch IB/CMSSW_10_6_X/gcc700.

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

vbfnlo.spec Outdated
@@ -0,0 +1,24 @@
### RPM external vbfnlo 3.0.0beta5
Source: http://www.itp.kit.edu/~vbfnloweb/archive/vbfnlo-%{realversion}.tgz
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Andrej-CMS , why not use https://github.com/vbfnlo/vbfnlo to download it? You can use

git+https://github.com/vbfnlo/%{n}.git?obj=master/v%{realversion}&export=%{n}-%{realversion}&output=/%{n}-%{realversion}.tgz

and then can do

%setup -q -n %{n}-%{realversion}

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dear @smuzaffar ,
I get now an error that states that the Manual.pdf file cannot be compiled from TeX file. Do we have in cmsdist some latex package that need to build first?
vbfnlobuild.txt

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@smuzaffar
do you have any idea in regards of the question above?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Andrej-CMS , I do not see build errors if I run autoreconf -vi before configure

@smuzaffar
Copy link
Contributor

@Andrej-CMS , do we have any tests in CMSSW to test hjets?

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 6, 2019

Pull request #4753 was updated.

@Andrej-CMS
Copy link
Contributor Author

@smuzaffar
thanks for your comments!
I changed to download from git. We don't have the hjets test yet. Do you want me to add a runTheMatrix command?

@smuzaffar
Copy link
Contributor

@Andrej-CMS , at least a unit test to make sure that our distribution of hjets is good/working

@Andrej-CMS
Copy link
Contributor Author

@smuzaffar
sure, I can add that. Could you maybe in the meantime try to build this here?
I had quite some issues building these packages, but this version here at least worked for me locally

@smuzaffar
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 6, 2019

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

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 6, 2019

-1

Tested at: 7caa456

  • Build:

I found compilation error when building:

+ CC=/build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/slc7_amd64_gcc700/external/gcc/7.0.0-omkpbe2/bin/gcc
++ which gfortran
+ FC=/build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/slc7_amd64_gcc700/external/gcc/7.0.0-omkpbe2/bin/gfortran
+ ./configure --prefix=/build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/tmp/BUILDROOT/608f29fd9a397d473ce59ff03edd9f22/opt/cmssw/slc7_amd64_gcc700/external/vbfnlo/3.0.0beta5 --enable-processes=vbf,hjjj FC=/build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/slc7_amd64_gcc700/external/gcc/7.0.0-omkpbe2/bin/gfortran FCFLAGS=-std=legacy
/build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/tmp/rpm-tmp.CtwoJG: line 34: ./configure: No such file or directory
error: Bad exit status from /build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/tmp/rpm-tmp.CtwoJG (%build)


RPM build errors:
Bad exit status from /build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/tmp/rpm-tmp.CtwoJG (%build)



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

@mrodozov
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 12, 2019

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

@cmsbuild
Copy link
Contributor

-1

Tested at: 7caa456

  • Build:

I found compilation error when building:

+ CC=/build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/slc7_amd64_gcc700/external/gcc/7.0.0-omkpbe2/bin/gcc
++ which gfortran
+ FC=/build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/slc7_amd64_gcc700/external/gcc/7.0.0-omkpbe2/bin/gfortran
+ ./configure --prefix=/build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/tmp/BUILDROOT/608f29fd9a397d473ce59ff03edd9f22/opt/cmssw/slc7_amd64_gcc700/external/vbfnlo/3.0.0beta5 --enable-processes=vbf,hjjj FC=/build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/slc7_amd64_gcc700/external/gcc/7.0.0-omkpbe2/bin/gfortran FCFLAGS=-std=legacy
/build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/tmp/rpm-tmp.nDzFao: line 34: ./configure: No such file or directory
error: Bad exit status from /build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/tmp/rpm-tmp.nDzFao (%build)


RPM build errors:
Bad exit status from /build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/tmp/rpm-tmp.nDzFao (%build)



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

vbfnlo.spec Outdated
CXX="$(which g++) -std=c++11"
CC="$(which gcc)"
FC="$(which gfortran)"

Copy link
Contributor

@smuzaffar smuzaffar Mar 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Andrej-CMS , as mentioned in https://github.com/vbfnlo/vbfnlo/blob/master/README.md , you need to run autoreconf -vi before running configure

@cmsbuild
Copy link
Contributor

cmsbuild commented May 20, 2019

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/348/console Started: 2019/05/20 15:33

@cmsbuild
Copy link
Contributor

Pull request #4753 was updated.

@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-ff8518/348/summary.html

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 0 differences found in the comparisons
  • DQMHistoTests: Total files compared: 33
  • DQMHistoTests: Total histograms compared: 3206828
  • DQMHistoTests: Total failures: 1
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 3206493
  • DQMHistoTests: Total skipped: 334
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 32 files compared)
  • Checked 137 log files, 14 edm output root files, 33 DQM output files

@smuzaffar
Copy link
Contributor

+externals
@alberto-sanchez,@perrozzi,@efeyazgan,@qliphy can you please sign this?

@qliphy
Copy link
Contributor

qliphy commented May 21, 2019

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_11_0_X/gcc700 IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2)

@smuzaffar
Copy link
Contributor

@fabiocos , this is ready to go in too.

@fabiocos
Copy link
Contributor

+1

@cmsbuild cmsbuild merged commit ab68e84 into cms-sw:IB/CMSSW_11_0_X/gcc700 May 21, 2019
@smuzaffar
Copy link
Contributor

@Andrej-CMS , HJets failed to compile with GCC 9

https://cmssdt.cern.ch/SDT/jenkins-artifacts/build-any-ib/CMSSW_11_0_X_2019-06-05-2300/slc7_amd64_gcc900/76593/external/hjets/1.2/log

  CXX      HJetsProcessInfo.lo
In file included from /build/cmsbld/jenkins/workspace/build-any-ib/w/slc7_amd64_gcc900/external/thepeg/2.1.5/include/ThePEG/Config/Complex.h:16,
                 from /build/cmsbld/jenkins/workspace/build-any-ib/w/slc7_amd64_gcc900/external/thepeg/2.1.5/include/ThePEG/Config/ThePEG.h:31,
                 from HJetsProcessInfo.h:13,
                 from HJetsProcessInfo.cc:11:
/build/cmsbld/jenkins/workspace/build-any-ib/w/slc7_amd64_gcc900/external/gcc/9.0.0-nmpfii/include/c++/9.1.1/complex: In instantiation of 'std::complex<double>& std::complex<double>::operator=(const std::complex<_Tp>&) [with _Tp = ThePEG::Qty<0, 0, 0>]':
HJetsProcessInfo.cc:240:93:   required from here
/build/cmsbld/jenkins/workspace/build-any-ib/w/slc7_amd64_gcc900/external/gcc/9.0.0-nmpfii/include/c++/9.1.1/complex:1319:19: error: 'const struct std::complex<ThePEG::Qty<0, 0, 0> >' has no member named '__rep'
 1319 |    _M_value = __z.__rep();
      |               ~~~~^~~~~
make[1]: *** [HJetsProcessInfo.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
In file included from /build/cmsbld/jenkins/workspace/build-any-ib/w/slc7_amd64_gcc900/external/thepeg/2.1.5/include/ThePEG/Config/Complex.h:16,
                 from /build/cmsbld/jenkins/workspace/build-any-ib/w/slc7_amd64_gcc900/external/thepeg/2.1.5/include/ThePEG/Config/ThePEG.h:31,
                 from /build/cmsbld/jenkins/workspace/build-any-ib/w/slc7_amd64_gcc900/external/thepeg/2.1.5/include/ThePEG/Interface/Interfaced.h:13,
                 from /build/cmsbld/jenkins/workspace/build-any-ib/w/slc7_amd64_gcc900/external/thepeg/2.1.5/include/ThePEG/Handlers/HandlerBase.h:13,
                 from /build/cmsbld/jenkins/workspace/build-any-ib/w/slc7_amd64_gcc900/external/thepeg/2.1.5/include/ThePEG/MatrixElement/Amplitude.h:14,
                 from /build/cmsbld/jenkins/workspace/build-any-ib/w/slc7_amd64_gcc900/external/herwigpp/7.1.5/include/Herwig/MatrixElement/Matchbox/Base/MatchboxAmplitude.h:15,
                 from AmplitudeBase.h:16,
                 from AmplitudeBase.cc:15:
/build/cmsbld/jenkins/workspace/build-any-ib/w/slc7_amd64_gcc900/external/gcc/9.0.0-nmpfii/include/c++/9.1.1/complex: In instantiation of 'std::complex<double>& std::complex<double>::operator=(const std::complex<_Tp>&) [with _Tp = ThePEG::Qty<0, 0, 0>]':
AmplitudeBase.cc:587:70:   required from here
/build/cmsbld/jenkins/workspace/build-any-ib/w/slc7_amd64_gcc900/external/gcc/9.0.0-nmpfii/include/c++/9.1.1/complex:1319:19: error: 'const struct std::complex<ThePEG::Qty<0, 0, 0> >' has no member named '__rep'
 1319 |    _M_value = __z.__rep();
      |               ~~~~^~~~~
make[1]: *** [AmplitudeBase.lo] Error 1

@smuzaffar
Copy link
Contributor

@fabiocos , @Andrej-CMS I am afraid we need to revert it back as it fails to build for GCC 9

@fabiocos
Copy link
Contributor

@Andrej-CMS I think that the code developers/maintainers should be notified about this issue, adapting also the extensions to the previous fix.

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

6 participants