-
Notifications
You must be signed in to change notification settings - Fork 185
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
Update to Rivet 3.1.2 and Yoda 1.8.3 #6028
Conversation
A new Pull Request was created by @intrepid42 (Markus Seidel) for branch IB/CMSSW_11_2_X/master. @cmsbuild, @smuzaffar, @mrodozov, @tulamor can you please review it and eventually sign? Thanks. |
please test |
The tests are being triggered in jenkins.
|
-1 Tested at: 0d6679d
I found compilation error when building: Hunk #2 succeeded at 227 (offset 27 lines). 1 out of 2 hunks FAILED -- saving rejects to file include/Rivet/AnalysisHandler.hh.rej patching file src/Core/AnalysisHandler.cc Hunk #1 succeeded at 69 (offset 7 lines). Hunk #2 succeeded at 79 (offset 7 lines). error: Bad exit status from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.lMXCgq (%prep) RPM build errors: Bad exit status from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.lMXCgq (%prep) You can see the results of the tests here: |
the patch needs adjustment :/ |
Pull request #6028 was updated. |
Done! (It worked locally with fuzz=1, and then I recreated the patch to work with fuzz=0) |
please test |
The tests are being triggered in jenkins.
|
-1 Tested at: 70a7db6
I found compilation error when building: make[2]: Leaving directory `/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/BUILD/slc7_amd64_gcc820/external/rivet/3.1.2-ccd3e1/rivet-3.1.2/analyses' make[2]: Leaving directory `/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/BUILD/slc7_amd64_gcc820/external/rivet/3.1.2-ccd3e1/rivet-3.1.2/analyses' make[2]: Leaving directory `/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/BUILD/slc7_amd64_gcc820/external/rivet/3.1.2-ccd3e1/rivet-3.1.2/analyses' make[1]: Leaving directory `/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/BUILD/slc7_amd64_gcc820/external/rivet/3.1.2-ccd3e1/rivet-3.1.2/analyses' make: *** [all-recursive] Error 1 error: Bad exit status from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.04fHex (%build) RPM build errors: Bad exit status from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.04fHex (%build) You can see the results of the tests here: |
Hi all, it turns out that this needs an update of fastjet-contrib to include the LundPlane plugin:
@smuzaffar @fabiocos I see you were the last ones to update fast-jet contrib. Would it be possible to go to the latest 1.044 there? |
I'll do it |
Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped) |
The tests are being triggered in jenkins.
|
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@smuzaffar, @mrodozov, @tulamor cms-sw/cmssw#30590 is ready. Can we merge this PR? |
yes, please :) |
+externals |
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_11_2_X/master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
Could this Rivet update be causing many failures in the NANO step along
? (followed up in cms-sw/cmssw#30836) |
Contains a couple of new features and analyses.
Hopefully no interface changes needed. Otherwise, unit tests should fail.