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
remove a bunch of unneeded Pythia6 configs #16578
Conversation
A new Pull Request was created by @kpedro88 (Kevin Pedro) for CMSSW_8_1_X. It involves the following packages: Configuration/Generator @cmsbuild, @govoni, @perrozzi, @thuer, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here #13028 |
@cmsbuild please test |
The tests are being triggered in jenkins. |
-1 Tested at: 3ae5369 You can see the results of the tests here: I found follow errors while testing this PR Failed tests: AddOn
I found errors in the following addon tests: cmsDriver.py TTbar_Tauola_13TeV_cfi -s GEN,SIM,DIGI,L1,DIGI2RAW --mc --scenario=pp -n 10 --conditions auto:run2_mc_GRun --relval 9000,50 --datatier "GEN-SIM-RAW" --eventcontent RAWSIM --customise=HLTrigger/Configuration/CustomConfigs.L1T --era Run2_2016 --magField 38T_PostLS1 --fileout file:RelVal_Raw_GRun_MC.root : FAILED - time: date Mon Nov 14 22:08:54 2016-date Mon Nov 14 22:08:00 2016 s - exit: 256 |
Comparison job queued. |
Pull request #16578 was updated. @perrotta, @smuzaffar, @Dr15Jones, @silviodonato, @perrozzi, @thuer, @dmitrijus, @Martin-Grunewald, @cmsbuild, @fwyzard, @vanbesien, @govoni, @davidlange6 can you please check and sign again. |
The AddOn test failure demonstrates that some of these configs might be used outside of relvals, so I checked for any usage of each file and updated that usage to the pythia8 version. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
Comparison is ready Alternative comparison was/were failed for workflow(s): |
Comparison job queued. |
+1 |
+1 |
+1 |
@davidlange6 @hengne requested followup to #16535. I went through and found all the Pythia6 configs in
Configuration/Generator
that had obvious Pythia8 counterparts (identical filenames except with either "TuneCUETP8M1" or "pythia8" in the newer version). I also checked that none of the removed files are used anywhere inConfiguration/PyReleaseValidation
.There are still a number of configs that don't have any Pythia8 counterpart. If PDMV wants to move all of those to Pythia8 as well, probably better to appoint someone to do it...