Skip to content
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

feat: add HEP cloud_pipeline stack #40932

Open
wants to merge 27 commits into
base: develop
Choose a base branch
from
Open

feat: add HEP cloud_pipeline stack #40932

wants to merge 27 commits into from

Conversation

wdconinc
Copy link
Contributor

@wdconinc wdconinc commented Nov 7, 2023

This PR adds a high energy physics (HEP) cloud pipeline stack. The goal is to include at least the core packages in HEP, to ensure they are tested when the underlying dependencies are modified or updated. For that reason, many of the variants have been enabled, even if they are not enabled by default.

TODO:

At this point it includes the following packages, but clearly could include a lot more (snapshot updated 2024-04-17):

  • acts +analysis +binaries +dd4hep +edm4hep +examples +fatras +geant4 +hepmc3 +identification +podio +pythia8 +python +tgeo
  • dd4hep +ddalign +ddcad +ddcond +dddetectors +dddigi +ddeve +ddg4 +ddrec +edm4hep +hepmc3 +lcio +utilityapps +xercesc
  • delphes +pythia8
  • edm4hep
  • fastjet
  • fjcontrib
  • gaudi +aida +examples +heppdt +xercesc ^gdb +python
  • geant4 +opengl +qt +threads ~vtk
  • hepmc
  • hepmc3 +interfaces +protobuf +python +rootio
  • herwig3 +njet +vbfnlo
  • lcio -examples +jar +rootdict # Note: lcio +examples ^ncurses -termlib, which leads to conflicts
  • lhapdf +python
  • madgraph5amc
  • opendatadetector
  • podio +rntuple +sio
  • py-uproot +lz4 +xrootd +zstd
  • pythia8 +evtgen +fastjet +hdf5 +hepmc +hepmc3 +lhapdf ~madgraph5amc +python +rivet -root # pythia8 and root circularly depend
  • rivet hepmc=3
  • root +davix +dcache +examples +fftw +fits +fortran +gdml +graphviz +gsl +http +math +minuit +mlp +mysql +opengl ~oracle +postgres +pythia6 +pythia8 +python +r +roofit +root7 +shadow +spectrum +sqlite +ssl +tbb +threads +tmva +unuran +vc +vdt +veccore +webgui +x +xml +xrootd
  • sherpa +analysis -blackhat +fastjet +gzip +hepmc2 +hepmc3 +hepmc3root +hztool +lhapdf +lhole +openloops +pythia ~python ~recola ~rivet +root +ufo
  • vecgeom +gdml +geant4 +root
  • xrootd +davix +http +krb5 +python +readline +scitokens-cpp

@spackbot-app spackbot-app bot added core PR affects Spack core functionality gitlab Issues related to gitlab integration labels Nov 7, 2023
@alalazo
Copy link
Member

alalazo commented Apr 15, 2024

I assume this is still a work in progress. Let me know when it's ready to be merged.

@wdconinc
Copy link
Contributor Author

Cherry-picked #43671 and #43672 while awaiting their merging...

@wdconinc
Copy link
Contributor Author

This finally builds the hep cloud pipeline. Remaining failure is the chai issue in https://spackpm.slack.com/archives/C02N33GM28H/p1713298552011109?thread_ts=1713298552.011109&cid=C02N33GM28H.

@wdconinc
Copy link
Contributor Author

@jmcarcell Do you want to review this and indicate what else would be useful to add here, based on what packages and variants you depend on in key4hep?

@alalazo At this point this is unify: false. Is there a preference for unify: when_possible? I don't see other cloud pipelines use when_possible; they only use false (most) or true (only in developer-tools).

Copy link
Contributor

@jmcarcell jmcarcell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me. I don't think I have anything to add related to the variants.

@wdconinc
Copy link
Contributor Author

Rebased to pick up #43738 fixing chai build.

@vvolkl
Copy link
Contributor

vvolkl commented Apr 28, 2024

Hi Wouter,

I think this is fantastic! Look good to me wrt. variants, I think @andresailer has also taken a look to check that this works for lcg releases.

The selection of packages seems good too. The most minmal stack that makes sense to me just needs to include dd4hep ( all variants enabled ), but this seems a good basis.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conflicts core PR affects Spack core functionality dependencies gitlab Issues related to gitlab integration update-package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants