Merge pull request #17 from waikato-ahuora-smart-energy-systems/fix-h…

…elmholtz-registered-components raise error if component not registered
waikato-ahuora-smart-energy-systems · Jan 6, 2025 · 2274477 · 2274477
2 parents ce4290f + 46514e4
commit 2274477
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/.github/workflows/python.yml b/.github/workflows/python.yml
@@ -28,7 +28,9 @@ jobs:
         python -m pip install --upgrade pip
         pip install flake8 pytest
         if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
-        idaes get-extensions
+        sudo apt-get update
+        sudo apt-get install liblapack3 liblapack-dev libgfortran5 libgomp1  # shared libraries for ipopt
+        idaes get-extensions --distro ubuntu2204
     - name: Lint with flake8
       run: |
         # stop the build if there are Python syntax errors or undefined names

diff --git a/property_packages/helmholtz/helmholtz_builder.py b/property_packages/helmholtz/helmholtz_builder.py
@@ -27,5 +27,5 @@ def build_helmholtz_package(compound_list: List[str]):
                                             phase_presentation=PhaseType.MIX,
                                             state_vars=StateVars.PH,
                                             amount_basis=AmountBasis.MOLE)
-
-
+        else:
+            raise ValueError(f"Compound {component} not found in Helmholtz registered components")