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
[GENERATORS] Update all the scripts to use python3 #37105
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37105/28600
|
A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for master. It involves the following packages:
@SiewYan, @mkirsano, @Saptaparna, @cmsbuild, @GurpreetSinghChahal, @alberto-sanchez can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@Dominic-Stafford: does this affect anything significantly regarding Herwig? Thanks! |
@Saptaparna I don't think so for now, it is included in the unit test. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-479fd8/22755/summary.html Comparison SummarySummary:
|
@Saptaparna , there is no python error/warning found in the tests [*],[**]. [*] https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-479fd8/22755/clang-logs/ |
@SiewYan , please check the unit tests e.g.
is there any unit test for testing |
by the way, do we know why https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-479fd8/22755/unitTests/src/GeneratorInterface/LHEInterface/test/testGeneratorInterfaceLHEInterface_TP/testing.log has some exceptions . These are already there in the IBs too. Are these expected and if not then why unit test is not failing and exiting with non-zero exit code? |
No, the only change on the Herwig side is to the parallelisation.py script, which isn't currently used for anything as far as I know. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-479fd8/22794/summary.html Comparison SummarySummary:
|
@smuzaffar thanks for catching this error, apparently there is a bug on the unitest code, the file format is different, cmsLHEtoEOSManager.py. I will address this later this week. |
@cms-sw/generators-l2 can you please review this? I think this looks good and does not break any thing new :-) |
@smuzaffar , sure no problem! i will work on this tiny issue from another PR. Thanks for pointing out though. |
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
Cleanup python2 usage