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

Align rrelu_with_noise schema to reflect noise mutation [branch] #8363

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

IvanKobzarev
Copy link
Collaborator

Creating instead of #8309

As fork branch can not be used in torch xla commit pin.

pytorch PR is pytorch/pytorch#138503

rrelu_with_noise actually mutates noise, but this was not reflected in schema.
As a result compilation did not capture its mutation.

This PR is to align xla override for rrelu_with_noise to remove const for noise argument

IvanKobzarev added a commit to pytorch/pytorch that referenced this pull request Dec 3, 2024
…tion in compile"


Rebase-copy of long standing already approved PR #138503 that was blocked on landing by xla build issues.


Got a new  PR with the same content (ghstack checkout was failing due to changed submodules)

Corresponding xla PR:
pytorch/xla#8363

cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx ipiszy yf225 chenyang78 kadeng muchulee8 ColinPeppler amjames desertfire chauhang aakhundov

[ghstack-poisoned]
IvanKobzarev added a commit to pytorch/pytorch that referenced this pull request Dec 3, 2024
Rebase-copy of long standing already approved PR #138503 that was blocked on landing by xla build issues.


Got a new  PR with the same content (ghstack checkout was failing due to changed submodules)

Corresponding xla PR:
pytorch/xla#8363

cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx ipiszy yf225 chenyang78 kadeng muchulee8 ColinPeppler amjames desertfire chauhang aakhundov

[ghstack-poisoned]
@qihqi
Copy link
Collaborator

qihqi commented Dec 3, 2024

Merging later after the torch side PR merges first, then torch_pin file should be removed.

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