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

[IPEX deprecation] Process to update upstream Pytorch pin and automation around it #1933

Closed
vlad-penkin opened this issue Aug 19, 2024 · 5 comments · Fixed by #2044, #2089, #2097, #2161 or #2219
Closed

Comments

@vlad-penkin
Copy link
Contributor

vlad-penkin commented Aug 19, 2024

Run validation worklfow for latest commit in main branch, which will include:

  • UT + Tutorials validation matrix - Supported Python versions x Driver Streams (Rolling, LTS)
  • Inductor for all supported Python versions on Rolling
  • E2E Accuracy LTS and Rolling for selected Python version
  • Benchmarks for selected Python version

Workflow run frequency

  • Monday and Wednesday or ad-hoc

Tracking

  • manually via GH issues We'll start from manual check and consider automation later

Baseline comparison with current UT/Tutorials and E2E results

  • We'll start from manual check and consider automation later
@pbchekin pbchekin self-assigned this Aug 20, 2024
@vlad-penkin vlad-penkin assigned kwasd and unassigned pbchekin Aug 20, 2024
@vlad-penkin vlad-penkin added this to the 7. CI milestone Aug 21, 2024
@vlad-penkin vlad-penkin linked a pull request Aug 29, 2024 that will close this issue
kwasd added a commit that referenced this issue Aug 30, 2024
#1933

---------

Co-autfhored-by: Pavel Chekin <[email protected]>
@vlad-penkin vlad-penkin linked a pull request Aug 30, 2024 that will close this issue
@vlad-penkin vlad-penkin linked a pull request Sep 3, 2024 that will close this issue
@vlad-penkin vlad-penkin reopened this Sep 5, 2024
@pbchekin
Copy link
Contributor

pbchekin commented Sep 8, 2024

Inductor for all supported Python versions on Rolling

Only one Python is fine. Also inductor tests can be a gateway for other tests: execute it first, then if successful execute rest of the tests.

@kwasd kwasd closed this as completed in 676c334 Sep 9, 2024
@kwasd
Copy link
Contributor

kwasd commented Sep 9, 2024

Done 3 of 4 (merged to llvm-target), but still need #1905 to schedule benchmarks for selected Python version: this adds pytorch_ref handling to the triton-benchmarks.yml.

@kwasd
Copy link
Contributor

kwasd commented Sep 9, 2024

Inductor for all supported Python versions on Rolling

Only one Python is fine. Also inductor tests can be a gateway for other tests: execute it first, then if successful execute rest of the tests.

See #2161

@vlad-penkin vlad-penkin linked a pull request Sep 9, 2024 that will close this issue
@kwasd
Copy link
Contributor

kwasd commented Sep 9, 2024

@kwasd
Copy link
Contributor

kwasd commented Sep 9, 2024

Will continue in #2172

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment