Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 2 additions & 3 deletions .github/workflows/build_windows_pytorch_wheels.yml
Original file line number Diff line number Diff line change
Expand Up @@ -117,9 +117,7 @@ jobs:
# TODO(amd-justchen): share with build_windows_packages.yml. Include in VM image? Dockerfile?
- name: Install requirements
run: |
# ninja pinned due to a bug in the 1.13.0 release:
# https://github.com/ninja-build/ninja/issues/2616
choco install --no-progress -y ninja --version 1.12.1
choco install --no-progress -y ninja --version 1.13.1
choco install --no-progress -y awscli
echo "$PATH;C:\Program Files\Amazon\AWSCLIV2" >> $GITHUB_PATH

Expand Down Expand Up @@ -161,6 +159,7 @@ jobs:
--pytorch-dir ${{ env.CHECKOUT_ROOT }}/torch ^
--pytorch-audio-dir ${{ env.CHECKOUT_ROOT }}/audio ^
--pytorch-vision-dir ${{ env.CHECKOUT_ROOT }}/vision ^
--enable-pytorch-flash-attention-windows ^
--clean ^
--output-dir ${{ env.PACKAGE_DIST_DIR }} ^
${{ env.optional_build_prod_arguments }}
Expand Down
28 changes: 6 additions & 22 deletions external-builds/pytorch/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -28,12 +28,12 @@ This incorporates advice from:

### Project and feature support status

| Project / feature | Linux support | Windows support |
| ------------------------ | ------------- | --------------------------------------------------------------------- |
| torch | ✅ Supported | ✅ Supported |
| torchaudio | ✅ Supported | ✅ Supported |
| torchvision | ✅ Supported | ✅ Supported |
| Flash attention (Triton) | ✅ Supported | 🟡 In progress ([#1040](https://github.com/ROCm/TheRock/issues/1040)) |
| Project / feature | Linux support | Windows support |
| ------------------------------ | ------------- | --------------- |
| torch | ✅ Supported | ✅ Supported |
| torchaudio | ✅ Supported | ✅ Supported |
| torchvision | ✅ Supported | ✅ Supported |
| Flash attention via [ao]triton | ✅ Supported | ✅ Supported |

### Supported PyTorch versions

Expand Down Expand Up @@ -95,22 +95,6 @@ for more background on these `rocm` packages.
> On Windows, when building with "--enable-pytorch-flash-attention-windows",
> Make sure to use [ninja 1.13.1](https://github.com/ninja-build/ninja/releases/tag/v1.13.1) or above.
>
> PyTorch builds aotriton locally, but without the kernel images.
> Make sure to copy the `aotriton.images` folder from an existing
> aotriton linux build (`<aotriton_build_dir>/lib/aotriton.images`) and copy
> that folder into your local pytorch lib directory: `<pytorch_dir>/torch/lib/`.
> This is a temporary measure for manually producing aotriton builds.
> NOTE: This will not work without the [corresponding patch](./patches/pytorch/main/pytorch/hipified/0004-Support-FLASH_ATTENTION-MEM_EFF_ATTENTION-via.-aotri.patch) for the main branch.
>
> On Windows, aotriton uses `dladdr`, which is implemented through
> [dlfcn-win32](https://github.com/dlfcn-win32/dlfcn-win32), which unfortunately
> uses `GetModuleFileNameA` (ANSI version) to get the base directory of
> `libaotriton.so`. This means, if `libaotriton.so` is put under a path with
> characters that cannot represented in current code page, the loading of GPU
> kernels will fail.
> See https://github.com/ROCm/aotriton/commit/e1be21d80b25f46139c2e3b4b0615e0279feccac
> For possible fixes. A proper fix is planned and will eventually be added.
>
> NOTE: If you use ccache and face "invalid argument" errors during the aotriton build,
> disable ccache and try again.

Expand Down
Loading