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

Install.md #246

Merged
merged 23 commits into from
Aug 7, 2024
Merged

Install.md #246

merged 23 commits into from
Aug 7, 2024

Conversation

justinlaughlin
Copy link
Contributor

Clarifying installation instructions. WIP

@sshiraiwa
Copy link
Member

A few comments

  1. --mfem-branch="master"is not wrong, but ```--mfem-branch=master``' works just fine
  2. PyMFEM does not build suitesparse nor cuda. They should not listed together with metis, hypre, gslib, libceed.
  3. It is good to mention minimum pip install. I usually do "pip install cmake, swig, numba-scipy" and "pip install mpi4py" for parallel.

@sshiraiwa sshiraiwa added this to the PyMFEM 4.7 milestone Aug 7, 2024
@sshiraiwa sshiraiwa mentioned this pull request Aug 7, 2024
12 tasks
justinlaughlin and others added 19 commits August 7, 2024 11:44
@justinlaughlin justinlaughlin marked this pull request as ready for review August 7, 2024 22:41
@sshiraiwa sshiraiwa self-assigned this Aug 7, 2024
@sshiraiwa sshiraiwa merged commit 3e68b55 into master Aug 7, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants