Assume Pythia83 for the master

Assume Pythia83 for the master

Tag @ewelina @akraszna @jchapman @cgutscho

This merge request affects 1 package:

• Generators/Pythia8_i

This merge request affects 5 files:

• Generators/Pythia8_i/CMakeLists.txt
• Generators/Pythia8_i/Pythia8_i/Pythia8_i.h
• Generators/Pythia8_i/Pythia8_i/UserHooksFactory.h
• Generators/Pythia8_i/cmake/FindHepMC3Pythia8Interface.cmake
• Generators/Pythia8_i/src/Pythia8_i.cxx

added Generators master review-pending-level-1 labels

The system determined that CI tests (with names matching "^CITest_Trigger.*$") are not needed for this code change. They are not run. This is not an indicator to restart the job.

CI Result SUCCESS (hash c7713fb0)

	Athena	AthGeneration
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 63326]

removed review-pending-level-1 label

added review-pending-expert label

changed the description

resolved all threads

@averbyts you've tagged several people, without any specific question, what feedback are you expecting, and from whom ? David as L2

Hi @droussea ,

we just discussed this MR with Ewelina, John and Attila. Christian was tagged as Pythia8 expert. I had a question for Attila, but I think the MR is OK even as is.

Andrii

Sorry for the delay... Yes, this looks fine. If we don't need to use that special/weird header out of HepMC3 anymore, then let's just remove the FindHepMC3PythiaInterface.cmake module. As I wrote in !59108 (merged), that setup was anyway not as robust as I would've liked it.

added review-approved label and removed review-pending-expert label

cc @gpanizzo @abuckley @mshapiro (Pythia8 experts)

resolved all threads

Dear @averbyts, all

everything looks fine from my side. Please go ahead, and let's monitor the next AthGeneration behavior keeping this change in mind.

Giancarlo

mentioned in commit 2673f14d

merged

added sweep:ignore label