Skip to content

Commit

Permalink
[Versioning] env-script for windows build (2) (#934)
Browse files Browse the repository at this point in the history
  • Loading branch information
vmoens authored Jul 30, 2024
1 parent ab58a0e commit 22685ea
Show file tree
Hide file tree
Showing 3 changed files with 103 additions and 1 deletion.
3 changes: 3 additions & 0 deletions .github/scripts/version_script.bat
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
@echo off
set TENSORDICT_BUILD_VERSION=0.5.0
echo TENSORDICT_BUILD_VERSION is set to %TENSORDICT_BUILD_VERSION%
2 changes: 1 addition & 1 deletion .github/workflows/build-wheels-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,4 +46,4 @@ jobs:
package-name: ${{ matrix.package-name }}
smoke-test-script: ${{ matrix.smoke-test-script }}
trigger-event: ${{ github.event_name }}
env-script: .github/scripts/version_script.sh
env-script: .github/scripts/version_script.bat
99 changes: 99 additions & 0 deletions .github/workflows/wheels-windows.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,99 @@
name: Wheels
on:
pull_request:
types: [opened, synchronize, reopened]
push:
branches:
- release/*

concurrency:
# Documentation suggests ${{ github.head_ref }}, but that's only available on pull_request/pull_request_target triggers, so using ${{ github.ref }}.
# On master, we want all builds to complete even if merging happens faster to make it easier to discover at which point something broke.
group: ${{ github.workflow }}-${{ github.ref == 'refs/heads/main' && format('ci-master-{0}', github.sha) || format('ci-{0}', github.ref) }}
cancel-in-progress: true

jobs:

build-wheel-windows:
runs-on: windows-latest
strategy:
matrix:
python_version: [["3.8", "3.8"], ["3.9", "3.9"], ["3.10", "3.10.3"], ["3.11", "3.11"], ["3.12", "3.12"]]
steps:
- name: Setup Python
uses: actions/setup-python@v2
with:
python-version: ${{ matrix.python_version[1] }}
- name: Checkout tensordict
uses: actions/checkout@v2
- name: Install PyTorch RC
shell: bash
run: |
python3 -mpip install torch --extra-index-url https://download.pytorch.org/whl/cpu
- name: Build wheel
shell: bash
run: |
python3 -mpip install wheel
TENSORDICT_BUILD_VERSION=0.5.0 python3 setup.py bdist_wheel
- name: Upload wheel for the test-wheel job
uses: actions/upload-artifact@v2
with:
name: tensordict-win-${{ matrix.python_version[0] }}.whl
path: dist/tensordict-*.whl
- name: Upload wheel for download
uses: actions/upload-artifact@v2
with:
name: tensordict-batch.whl
path: dist/*.whl

test-wheel-windows:
needs: build-wheel-windows
strategy:
matrix:
python_version: ["3.8", "3.9", "3.10", "3.11", "3.12" ]
runs-on: windows-latest
steps:
- name: Setup Python
uses: actions/setup-python@v2
with:
python-version: ${{ matrix.python_version }}
- name: Checkout tensordict
uses: actions/checkout@v2
- name: Install PyTorch RC
shell: bash
run: |
python3 -mpip install torch torchvision --extra-index-url https://download.pytorch.org/whl/cpu
- name: Upgrade pip
shell: bash
run: |
python3 -mpip install --upgrade pip
- name: Install test dependencies
shell: bash
run: |
python3 -mpip install numpy pytest pytest-cov codecov unittest-xml-reporting pillow>=4.1.1 scipy av networkx expecttest pyyaml
- name: Download built wheels
uses: actions/download-artifact@v2
with:
name: tensordict-win-${{ matrix.python_version }}.whl
path: wheels
- name: Install built wheels
shell: bash
run: |
python3 -mpip install wheels/*
- name: Log version string
shell: bash
run: |
# Avoid ambiguity of "import tensordict" by deleting the source files.
rm -rf tensordict/
python -c "import tensordict; print(tensordict.__version__)"
- name: Run tests
shell: bash
run: |
set -e
export IN_CI=1
mkdir test-reports
python -m torch.utils.collect_env
python -c "import tensordict; print(tensordict.__version__)"
EXIT_STATUS=0
pytest test/smoke_test.py -v --durations 200
exit $EXIT_STATUS

1 comment on commit 22685ea

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'CPU Benchmark Results'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 2.

Benchmark suite Current: 22685ea Previous: ab58a0e Ratio
benchmarks/common/common_ops_test.py::test_membership_stacked_nested_last 78734.35655215516 iter/sec (stddev: 7.045546056432869e-7) 252736.87756085233 iter/sec (stddev: 5.363273959416475e-7) 3.21
benchmarks/common/common_ops_test.py::test_membership_stacked_nested_leaf_last 78592.45558982134 iter/sec (stddev: 7.621893597490822e-7) 254420.900815467 iter/sec (stddev: 3.3076272345179124e-7) 3.24

This comment was automatically generated by workflow using github-action-benchmark.

CC: @vmoens

Please sign in to comment.