Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Windows CI #1028

Open
wants to merge 66 commits into
base: main
Choose a base branch
from
Open

Add Windows CI #1028

wants to merge 66 commits into from

Conversation

RUIJIEZHONG66166
Copy link
Contributor

No description provided.

@@ -35,6 +35,14 @@ jobs:
abi: 0
ut: op_extended
runner: linux.idc.xpu

preci-windows-build:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
preci-windows-build:
preci-windows:

Copy link
Contributor

Choose a reason for hiding this comment

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

Rename to _winodws_ut.yml

type: string
default: ''
description: UT scope. `op_regression,op_regression_dev1,op_extended,op_ut,torch_xpu` Delimiter is comma
abi:
Copy link
Contributor

Choose a reason for hiding this comment

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

Windows no abi options

runner:
required: true
type: string
default: 'linux.idc.xpu'
Copy link
Contributor

Choose a reason for hiding this comment

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

change the default value for windows

USE_XPU: 1

jobs:
Torch-XPU-Windows-Build:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Torch-XPU-Windows-Build:
Torch-XPU-Windows-UT:

Comment on lines 103 to 109
if ${{ inputs.abi }} == '0' (
set _GLIBCXX_USE_CXX11_ABI=0
echo "ABI=0"
) else (
set _GLIBCXX_USE_CXX11_ABI=1
echo "ABI=1"
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
if ${{ inputs.abi }} == '0' (
set _GLIBCXX_USE_CXX11_ABI=0
echo "ABI=0"
) else (
set _GLIBCXX_USE_CXX11_ABI=1
echo "ABI=1"
)

)
cd ..
Xcopy ..\torch-xpu-ops third_party\torch-xpu-ops /E/H/Y/F/I
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Add one more step to call install_xpu.bat

call "C:\ProgramData\miniforge3\Scripts\activate.bat"
call conda activate windows_ci
call "C:\Program Files (x86)\Intel\oneAPI\pytorch-gpu-dev-0.5\oneapi-vars.bat"
call "C:\Program Files (x86)\Intel\oneAPI\ocloc\2024.2\env\vars.bat"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
call "C:\Program Files (x86)\Intel\oneAPI\ocloc\2024.2\env\vars.bat"

call "C:\ProgramData\miniforge3\Scripts\activate.bat"
call conda activate windows_ci
call "C:\Program Files (x86)\Intel\oneAPI\pytorch-gpu-dev-0.5\oneapi-vars.bat"
call "C:\Program Files (x86)\Intel\oneAPI\ocloc\2024.2\env\vars.bat"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
call "C:\Program Files (x86)\Intel\oneAPI\ocloc\2024.2\env\vars.bat"

@chuanqi129
Copy link
Contributor

Hi @RUIJIEZHONG66166 , Please change the windows ci trigger condition as PR with windows label, you can follow the method in https://stackoverflow.com/questions/62325286/run-github-actions-when-pull-requests-have-a-specific-label.
And I saw there are 43 UT failures, some are env issue, please try to figure out them and filter them and invite @daisyden help to double check them.

@RUIJIEZHONG66166 RUIJIEZHONG66166 added os: Windows Windows Platform windows_ci and removed os: Windows Windows Platform labels Dec 17, 2024
@RUIJIEZHONG66166 RUIJIEZHONG66166 force-pushed the ruijie/add_windows_ci branch 2 times, most recently from 4951982 to 8a9d8a7 Compare December 25, 2024 01:22
@RUIJIEZHONG66166 RUIJIEZHONG66166 force-pushed the ruijie/add_windows_ci branch 2 times, most recently from 9c0ba4f to 60006c5 Compare December 27, 2024 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants