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

Repair pipeline #81

Merged
merged 1 commit into from
Dec 15, 2022
Merged

Conversation

tbody-cfs
Copy link
Contributor

The pipelines are currently failing due to a missing import of _version.

Replace the version variable with one from importlib.metadata

@tbody-cfs
Copy link
Contributor Author

Actually, this seems to be due to a failing install of HDF5

@tbody-cfs tbody-cfs closed this Dec 15, 2022
@tbody-cfs
Copy link
Contributor Author

running build_ext
         Loading library to get build settings and version: libhdf5.so
         error: Unable to load dependency HDF5, make sure HDF5 is installed properly
         error: libhdf5.so: cannot open shared object file: No such file or directory
         [end of output]
   
     note: This error originates from a subprocess, and is likely not a problem with pip.
     ERROR: Failed building wheel for h5py
   ERROR: Could not build wheels for h5py, which is required to install pyproject.toml-based projects

@tbody-cfs tbody-cfs reopened this Dec 15, 2022
@tbody-cfs tbody-cfs changed the title Replace __version__ with importlib.metadata Repair pipeline Dec 15, 2022
@tbody-cfs tbody-cfs marked this pull request as draft December 15, 2022 21:11
@tbody-cfs
Copy link
Contributor Author

I'll look at how to get HDF5 working. Could we add a conda install in the actions?

@tbody-cfs tbody-cfs marked this pull request as ready for review December 15, 2022 22:12
@tbody-cfs
Copy link
Contributor Author

It works! A big thanks to @istevenson-cfs for helping to figure this out!

@bendudson
Copy link
Collaborator

Awesome! Thanks, and nice work @tbody-cfs and @istevenson-cfs !

@bendudson bendudson merged commit d2713b1 into freegs-plasma:master Dec 15, 2022
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