Skip to content

Commit

Permalink
conda
Browse files Browse the repository at this point in the history
  • Loading branch information
Borda committed Jun 29, 2020
1 parent d7cf484 commit 40f3c19
Showing 1 changed file with 4 additions and 10 deletions.
14 changes: 4 additions & 10 deletions .github/workflows/pt-testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ jobs:
update-conda: true
python-version: ${{ matrix.python-version }}
conda-channels: anaconda, conda-forge
activate-conda: true

# TODO: add conda/pip caching
# https://github.com/actions/cache/blob/master/examples.md#multiple-oss-in-a-workflow
Expand All @@ -54,13 +55,6 @@ jobs:
# restore-keys: |
# ${{ runner.os }}-${{ matrix.python-version }}-${{ matrix.pytorch-version }}-pip-

- name: Create conda
run: |
conda init bash
# conda create --name ci-env
# conda activate ci-env
shell: bash

- name: Install stable PyTorch version
if: matrix.pytorch-version <= 1.6
run: |
Expand All @@ -76,7 +70,7 @@ jobs:
- name: Install dependencies
run: |
# python -m pip install --upgrade --user pip
HOROVOD_BUILD_ARCH_FLAGS="-mfma" conda env update --file environment.yml # --name ci-env
HOROVOD_BUILD_ARCH_FLAGS="-mfma" conda env update --file environment.yml --name base
python --version
conda info
conda list
Expand All @@ -85,7 +79,7 @@ jobs:
- name: Reinstall Horovod if necessary
if: runner.os != 'windows' && matrix.python-version != '3.8'
run: |
# conda activate ci-env
# conda activate base
HOROVOD_BUILT=$(python -c "import horovod.torch; horovod.torch.nccl_built(); print('SUCCESS')")
if [[ $HOROVOD_BUILT != "SUCCESS" ]]; then
pip uninstall -y horovod
Expand All @@ -103,7 +97,7 @@ jobs:

- name: Tests
run: |
# conda activate ci-env
# conda activate base
# NOTE: run coverage on tests does not propagare faler status for Win, https://github.com/nedbat/coveragepy/issues/1003
python -m pytest pytorch_lightning tests pl_examples -v --junitxml=junit/test-results-${{ runner.os }}-${{ matrix.python-version }}-${{ matrix.requires }}.xml
# coverage report
Expand Down

0 comments on commit 40f3c19

Please sign in to comment.