Skip to content

Commit 1758b79

Browse files
authored
Merge pull request #48 from tadkollar/mumpsfix
Uses newer conda packages for MUMPS and fixed the MacOS test workflow
2 parents 9070b60 + 7f90015 commit 1758b79

File tree

4 files changed

+18
-536
lines changed

4 files changed

+18
-536
lines changed

.github/workflows/test_workflow.yml

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -28,15 +28,16 @@ jobs:
2828
PAROPT: true
2929
SNOPT: 7.7
3030

31-
# # test baseline versions on MacOS
32-
# - NAME: MacOS Baseline
33-
# OS: macos-latest
34-
# PY: '3.10'
35-
# NUMPY: 1.22
36-
# SCIPY: 1.7
37-
# PYOPTSPARSE: 'v2.8.3'
38-
# PAROPT: true
39-
# SNOPT: 7.7
31+
# test baseline versions on MacOS
32+
- NAME: MacOS Baseline
33+
OS: macos-latest
34+
PY: '3.11'
35+
NUMPY: 1
36+
SCIPY: 1
37+
MPI4PY: true
38+
PYOPTSPARSE: 'latest'
39+
PAROPT: true
40+
SNOPT: 7.7
4041

4142
# test latest versions
4243
- NAME: Ubuntu Latest
@@ -192,15 +193,17 @@ jobs:
192193
193194
build_pyoptsparse -v $BRANCH $PAROPT $SNOPT $NO_IPOPT $LINEAR_SOLVER
194195
195-
echo "BRANCH=$BRANCH" >> $GITHUB_ENV
196+
echo "BRANCH=${BRANCH}" >> $GITHUB_ENV
196197
197198
- name: Run tests
198199
run: |
199200
python -m pip install testflo parameterized six
201+
conda install -y -q gettext
200202
201203
echo "============================================================="
202204
echo "Run tests from pyoptsparse repository"
203205
echo "============================================================="
206+
unset DYLD_LIBRARY_PATH
204207
git clone $BRANCH https://github.com/mdolab/pyoptsparse
205208
cd pyoptsparse/test*/
206209
testflo --pre_announce --show_skipped .

__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
__version__ = '2.0.4'
1+
__version__ = '2.0.5'

build_pyoptsparse.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -762,7 +762,9 @@ def install_mumps():
762762
""" Install MUMPS either through conda or building. """
763763
if allow_install_with_conda() and opts['force_build'] is False:
764764
try:
765-
install_conda_pkg('mumps')
765+
install_conda_pkg('mumps-include')
766+
install_conda_pkg('mumps-seq')
767+
install_conda_pkg('mumps-mpi')
766768
return
767769
except Exception as e:
768770
try_fallback('MUMPS', e)
@@ -1009,7 +1011,7 @@ def uninstall_conda_pkgs():
10091011
""" Attempt to remove packages previously installed by conda. """
10101012

10111013
if conda_is_active():
1012-
for pkg in ['ipopt','mumps','metis']:
1014+
for pkg in ['ipopt','mumps','mumps-include','mumps-seq','mumps-mpi','metis']:
10131015
note(f"Removing {pkg.upper()} conda package")
10141016
run_cmd(cmd_list=[opts['conda_cmd'],'uninstall','-y',pkg], do_check=False)
10151017
note_ok()

0 commit comments

Comments
 (0)