Skip to content

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

Merged
IvanKobzarev merged 1 commit intomasterfrom
rrelu_with_noise
Dec 4, 2024
Merged

Align rrelu_with_noise schema to reflect noise mutation [branch]#8363
IvanKobzarev merged 1 commit intomasterfrom
rrelu_with_noise

Conversation

@IvanKobzarev
Copy link
Copy Markdown
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 IvanKobzarev requested a review from qihqi November 7, 2024 21:41
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
Copy Markdown
Collaborator

qihqi commented Dec 3, 2024

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

pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request Dec 4, 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

Pull Request resolved: #141867
Approved by: https://github.com/bdhirsh
@IvanKobzarev IvanKobzarev merged commit 73f54ba into master Dec 4, 2024
pobin6 pushed a commit to pobin6/pytorch that referenced this pull request Dec 5, 2024
…1867)

Rebase-copy of long standing already approved PR pytorch#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

Pull Request resolved: pytorch#141867
Approved by: https://github.com/bdhirsh
AmdSampsa pushed a commit to AmdSampsa/pytorch that referenced this pull request Dec 9, 2024
…1867)

Rebase-copy of long standing already approved PR pytorch#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

Pull Request resolved: pytorch#141867
Approved by: https://github.com/bdhirsh
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