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

Add the pyhepmc package to the stack #672

Merged
merged 3 commits into from
Nov 28, 2024
Merged

Add the pyhepmc package to the stack #672

merged 3 commits into from
Nov 28, 2024

Conversation

jmcarcell
Copy link
Member

@BrieucF
This lets me do import pyhepmc without issues

@BrieucF
Copy link
Contributor

BrieucF commented Nov 20, 2024

Hi Juan, thanks! I realize that this is the version from scikit-learn, the Python bindings from HepMC3 are retrieved with import pyHepMC3 which, as you spotted, only works after

export PYTHONPATH=/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-09-21/x86_64-almalinux9-gcc14.2.0-opt/hepmc3/3.3.0-c2oek4/lib/python3.11/site-packages:$PYTHONPATH

can we also fix this?

@jmcarcell
Copy link
Member Author

Yes, that's independent of this. So is this package not needed and by pyhepmc you meant the python bindings of HepMC3 instead of the scikit package?

@BrieucF
Copy link
Contributor

BrieucF commented Nov 20, 2024

I have tried both pyhepmc and pyHepMC3, I think I will go for pyHepMC3 for the final solution but I can't say for sure now

@jmcarcell jmcarcell enabled auto-merge (squash) November 28, 2024 13:44
@jmcarcell jmcarcell merged commit 1429e2d into main Nov 28, 2024
5 of 7 checks passed
@jmcarcell jmcarcell deleted the pyhepmc branch November 28, 2024 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants