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

Quick fix for installation issues #21

Merged
merged 1 commit into from
Dec 2, 2020

Conversation

smhooten
Copy link
Contributor

  1. Fix of install.py to correct a recent update of the boost geometry repo which breaks c++11 compilation. This hard-codes a direct download to boost version 1.73.0.
  2. Hard-coded the versions of petsc4py and slepc4py that are retrieved in setup.py to fix an incompatibility with the versions of petsc and slepc that were installed from source in install.py. An alternate installation scheme might be preferable, but works for now.
  3. Minor bug fix in emopt/fdtd.py to correct the w_pml setter.

Tested on CentOS 7 with Python 2.7 and Python 3.8

… repo which breaks c++11 compilation. This hard-codes a direct download to boost version 1.73.0. 2. Hard-coded the versions of petsc4py and slepc4py that are retrieved in setup.py to fix an incompatibility with the versions of petsc and slepc that were installed from source in install.py. An alternate installation scheme might be preferable, but works for now. 3. Minor bug fix in emopt/fdtd.py to correct the w_pml setter
@anstmichaels anstmichaels merged commit 60fd236 into anstmichaels:master Dec 2, 2020
@anstmichaels
Copy link
Owner

Thanks for the fixes.

In the future, I think I will do away with install_deps.py all together. Instead, compatible versions of boost and eigen can now be installed from pretty much all of the mainstream linux package managers. We will rely on petsc4py and slepc4py to compile PETSc and SLEPc. Ill implement and test on the develop branch.

@smhooten smhooten deleted the install_fix branch January 1, 2021 00:20
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.

None yet

2 participants