Skip to content

Commit

Permalink
Revert "Validate pypi build only for release (#1623)"
Browse files Browse the repository at this point in the history
This reverts commit 4db3d68.
  • Loading branch information
atalman committed Dec 11, 2023
1 parent bfca402 commit f967d28
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 4 deletions.
9 changes: 7 additions & 2 deletions .github/scripts/validate_pipy.sh
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,14 @@ fi

if [[ ${TORCH_ONLY} == 'true' ]]; then
TEST_SUFFIX=" --package torchonly"
pip3 install torch${RELEASE_SUFFIX}
pip3 install --pre torch${RELEASE_SUFFIX} --extra-index-url "https://download.pytorch.org/whl/${MATRIX_CHANNEL}/${MATRIX_DESIRED_CUDA}_pypi_cudnn"
else
pip3 install torch${RELEASE_SUFFIX} torchvision torchaudio
if [[ ${MATRIX_CHANNEL} != "release" ]]; then
pip3 install --pre torch${RELEASE_SUFFIX} --extra-index-url "https://download.pytorch.org/whl/${MATRIX_CHANNEL}/${MATRIX_DESIRED_CUDA}_pypi_cudnn"
pip3 install --pre torchvision torchaudio --extra-index-url "https://download.pytorch.org/whl/${MATRIX_CHANNEL}/${MATRIX_DESIRED_CUDA}"
else
pip3 install torch${RELEASE_SUFFIX} torchvision torchaudio
fi
fi

python ./test/smoke_test/smoke_test.py ${TEST_SUFFIX} --runtime-error-check disabled
Expand Down
7 changes: 5 additions & 2 deletions .github/workflows/validate-linux-binaries.yml
Original file line number Diff line number Diff line change
Expand Up @@ -91,9 +91,12 @@ jobs:
cat release_matrix.json
# Special case PyPi installation package. And Install of PyPi package via poetry
if [[ ${MATRIX_PACKAGE_TYPE} == "manywheel" && ${MATRIX_GPU_ARCH_VERSION} == "12.1" && ${MATRIX_CHANNEL} == "release"]]; then
if [[ ${MATRIX_PACKAGE_TYPE} == "manywheel" && ${MATRIX_GPU_ARCH_VERSION} == "12.1" ]]; then
source ./.github/scripts/validate_pipy.sh
source ./.github/scripts/validate_poetry.sh
if [[ ${MATRIX_CHANNEL} == "release" ]]; then
source ./.github/scripts/validate_poetry.sh
fi
fi
# Standart case: Validate binaries
Expand Down

0 comments on commit f967d28

Please sign in to comment.