Skip to content

Commit

Permalink
[r2] CI: fix Python workflow (#4364)
Browse files Browse the repository at this point in the history
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- **Chores**
- Updated the workflow configuration for testing Python dependencies to
improve clarity and installation steps.
- Adjusted installation commands for `mpi4py` and `horovod` to
streamline the process.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
  • Loading branch information
njzjz authored Nov 15, 2024
1 parent 440bb82 commit aef3532
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion .github/workflows/test_python.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,9 @@ jobs:
# changes, setting `TENSORFLOW_ROOT`.
TENSORFLOW_VERSION: ${{ matrix.tf }}
DP_BUILD_TESTING: 1
- run: source/install/uv_with_retry.sh pip install --system --no-build-isolation horovod mpi4py
- run: |
source/install/uv_with_retry.sh pip install --system mpi4py
source/install/uv_with_retry.sh pip install --system --no-build-isolation horovod
if: ${{ matrix.tf == '' }}
env:
HOROVOD_WITH_TENSORFLOW: 1
Expand Down

0 comments on commit aef3532

Please sign in to comment.