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

Test redhat installation #111

Merged
merged 12 commits into from
Apr 26, 2024
Merged

Test redhat installation #111

merged 12 commits into from
Apr 26, 2024

Conversation

jorgensd
Copy link
Owner

Try to uncover what is wrong with #106 (comment)

@jorgensd jorgensd changed the base branch from main to release April 25, 2024 19:43
@jorgensd
Copy link
Owner Author

@gonsie I've set up a RedHat runner to test the code, and I cannot reproduce the error message you are observing.

I also added some installation notes regarding adios2 (and how it needs to be configured with MPI).

@jorgensd jorgensd merged commit 7bf967c into release Apr 26, 2024
12 checks passed
jorgensd added a commit that referenced this pull request Apr 27, 2024
* Revert 0.8 API changes for release

* Test against 0.7.3 image

* Update citation file and pyproject.toml

* Update workflows

* Correct version in readme

* Test redhat installation (#111)

* Test package on redhat

* Remove default build from matrix as redhat image does not have adios2 installed

* Re-introduce type hints from typing

* Add back adios2 to the Python dependencies

* Try setting working directory in test

* Remove petsc arch

* try fixing petsc dir again

* Install adios2 from source as it requires MPI build. Set pythonpath in parent workflow

* Add notes about adios2 installation

* Use newest install-dolfinx action

* Remove adios2 from deps as the pypi version is incompatible with package (no MPI support)
@jorgensd jorgensd deleted the dokken/test-redhat branch April 30, 2024 21:59
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.

1 participant