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
Rivet 2.6.1 + Yoda 1.7.4 #4427
Rivet 2.6.1 + Yoda 1.7.4 #4427
Conversation
A new Pull Request was created by @intrepid42 (Markus Seidel) for branch IB/CMSSW_10_4_X/gcc700. @cmsbuild, @smuzaffar, @gudrutis, @mrodozov can you please review it and eventually sign? Thanks. |
please test |
The tests are being triggered in jenkins. |
-1 Tested at: 3643de9 You can see the results of the tests here: I found follow errors while testing this PR Failed tests: RelVals
When I ran the RelVals I found an error in the following worklfows: runTheMatrix-results/1325.7_TTbar_13_94Xv2NanoAODINPUT+TTbar_13_94Xv2NanoAODINPUT+NANOEDMMC2017_94XMiniAODv2+HARVESTNANOAODMC2017_94XMiniAODv2/step2_TTbar_13_94Xv2NanoAODINPUT+TTbar_13_94Xv2NanoAODINPUT+NANOEDMMC2017_94XMiniAODv2+HARVESTNANOAODMC2017_94XMiniAODv2.log |
Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped) |
Hi, I see that the test fails on compilation warnings. Will fix that in a CMSSW pull request. |
workflow 1325.7 also fails due to 'static TLS' issue. |
Ah, I see. We run the Rivet-based ParticleLevelProducer for NanoAod there... So no reliable behavior on slc6 :( |
What's missing for this PR ? |
please test |
Comparison is ready Comparison Summary:
|
+1 |
Comparison job queued. |
@intrepid42 , tests passed for both slc6 and slc7. Can you please a cmssw PR to fix these warnings |
Comparison is ready Comparison Summary:
|
@smuzaffar Yes, please see cms-sw/cmssw#25817 |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+externals |
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_10_5_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) |
+1 @smuzaffar I understand that this version now passes also the test on slc6 (contrary to what initially stated in the explanation of this PR). There is already a fix at CMSSW level for the compilation warning, so let's test this in the IB. @intrepid42 we have a limited test in generatorInterface/RivetInterface, in case it could be expanded with more analyses testing more features, do you see this as interesting? |
Open issue with loading library, see #3679
slc6+slc7--> acceptable as slc6 will be phased out?
Other fixes: