-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Drop PhotosInterface, support for tauola 27.x from TauolaInterface #37616
Drop PhotosInterface, support for tauola 27.x from TauolaInterface #37616
Conversation
please test |
A new Pull Request was created by @iarspider for CMSSW_12_4_DEVEL_X. It involves the following packages:
@alberto-sanchez, @SiewYan, @mkirsano, @Saptaparna, @GurpreetSinghChahal can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-ab67de/24004/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Comparison SummarySummary:
|
please test with #37612,cms-sw/cmsdist#7807 |
please test with cms-sw/cmsdist#7807 |
please test with cms-sw/cmsdist#7807,cms-sw/cmsdist#7806 |
1 similar comment
please test with cms-sw/cmsdist#7807,cms-sw/cmsdist#7806 |
please test with cms-sw/cmsdist#7807,cms-sw/cmsdist#7806,#37612 |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-ab67de/24091/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
@cms-sw/generators-l2 , I suggest that we merge this (along with cmsdist PRs) in DEVEL IBs to test full runTheMatrix. If DEVEL IBs show no failure then we can backport these for |
please test with cms-sw/cmsdist#7807 |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-ab67de/24167/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
@cms-sw/generators-l2 do you have any objection in merging these PRs in the devel branch (not yet in the main release)? |
No objections. +1 |
merge |
Original PRs: cms-sw#37610, cms-sw#37611, cms-sw#37612, cms-sw#37616; cmsdist PR: todo
PR description:
Drop PhotosInterface due to removal of legacy MC generators (cms-sw/cmsdist#7763).