Skip to content

Companion PR for pytorch/pytorch#106272#5376

Closed
nkaretnikov wants to merge 1 commit intomasterfrom
pytorch-106272
Closed

Companion PR for pytorch/pytorch#106272#5376
nkaretnikov wants to merge 1 commit intomasterfrom
pytorch-106272

Conversation

@nkaretnikov
Copy link
Copy Markdown
Collaborator

No description provided.

@nkaretnikov
Copy link
Copy Markdown
Collaborator Author

Added meta for median in pytorch/pytorch#106272, which affected the counters.

@nkaretnikov nkaretnikov requested a review from JackCaoG July 31, 2023 03:00
@nkaretnikov
Copy link
Copy Markdown
Collaborator Author

Forgot to rebase on origin/master, fixed now.

Copy link
Copy Markdown
Collaborator

@JackCaoG JackCaoG left a comment

Choose a reason for hiding this comment

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

oh so the actual fix should be use a different op that pytorch/xla does not support. The purpose of this test is to test the fallback. @wonjoolee95 can you help on finding a new op that will fallback?

@nkaretnikov
Copy link
Copy Markdown
Collaborator Author

@JackCaoG @wonjoolee95 How about aten._foobar? It's an aux function that's used for some tests. Will it work for your usecase?

@JackCaoG
Copy link
Copy Markdown
Collaborator

JackCaoG commented Aug 1, 2023

yea any op that will fallback in pytorch/xla will do I believe

@wonjoo-wj
Copy link
Copy Markdown
Collaborator

Thanks @nkaretnikov, aten::_foobar is perfect. We'll merge #5393 as soon as its CI is green and your pytorch/pytorch#106272 should be unblocked.

@nkaretnikov
Copy link
Copy Markdown
Collaborator Author

@wonjoolee95 sounds good, closing this one, then!

@nkaretnikov nkaretnikov closed this Aug 2, 2023
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.

3 participants