Skip to content

Commit

Permalink
pybamm-team#3558 Remove test command for now
Browse files Browse the repository at this point in the history
Fails in CI on Windows but the wheel does contain a .pyd file, so it passes on Windows locally.

Another reason is that `casadi` cannot be imported on manylinux2014 containers at this moment
  • Loading branch information
agriyakhetarpal committed Nov 27, 2023
1 parent 4b426f5 commit 28af076
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/publish_pypi.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,6 @@ jobs:
CIBW_ENVIRONMENT: 'PYBAMM_USE_VCPKG=ON VCPKG_ROOT_DIR=C:\vcpkg VCPKG_DEFAULT_TRIPLET=x64-windows-static-md VCPKG_FEATURE_FLAGS=manifests,registries CMAKE_GENERATOR="Visual Studio 17 2022" CMAKE_GENERATOR_PLATFORM=x64'
CIBW_ARCHS: "AMD64"
CIBW_BEFORE_BUILD: python -m pip install setuptools wheel
CIBW_TEST_COMMAND: python -c 'import pybamm; print(pybamm.have_idaklu())' | grep 'True'

- name: Upload Windows wheels
uses: actions/upload-artifact@v3
Expand Down Expand Up @@ -112,7 +111,6 @@ jobs:
python -m pip install --upgrade cmake casadi setuptools wheel && scripts/fix_suitesparse_rpath_mac.sh
CIBW_REPAIR_WHEEL_COMMAND_MACOS: >
delocate-listdeps {wheel} && delocate-wheel -v -w {dest_dir} {wheel}
CIBW_TEST_COMMAND: python -c "import pybamm; print(pybamm.have_idaklu())" | grep "True"
CIBW_SKIP: "pp* *musllinux*"

- name: Upload wheels
Expand Down

0 comments on commit 28af076

Please # to comment.