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

Final changes necessary to auto-update PyTorch version #1438

Merged
merged 2 commits into from
Sep 29, 2022

Conversation

ashay
Copy link
Collaborator

@ashay ashay commented Sep 29, 2022

This PR contains two commits. The first commit pushes the PyTorch version update directly to the main branch, so that we avoid the PR + CI + Approval + Merging steps. The second commit schedules this workflow to be scheduled daily.


I checked that the workflow runs as expected and pushes the change to the main branch if the tests pass (see commit 15e1c45).

This avoids the need to create, approve, and merge a separate PR, in
addition to avoiding unnecessary CI runs for the PyTorch version update.
This patch schedules the RollPyTorch action to be run at noon UTC, which
roughly corresponds to 4am Pacific Time.  We pick this time since the
commit for PyTorch nightly releases are picked just after midnight
Pacific Time and the nightly release artifacts are produced in about 2
to 3 hours after the commit is picked.
Copy link
Collaborator

@powderluv powderluv left a comment

Choose a reason for hiding this comment

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

thank you

@ashay ashay merged commit cf41a25 into main Sep 29, 2022
@ashay ashay deleted the ashay/push-pytorch-update branch September 29, 2022 22:15
qedawkins pushed a commit to nod-ai/torch-mlir that referenced this pull request Oct 3, 2022
Move `onnx-mlir` version to `0.3.0` from `0.2.0`. `1.0.0` was a mistake.

Signed-off-by: Whitney Tsang <whitneyt@ca.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants