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

[py][vulkan][reland] Add is_vulkan to py api, add vulkan to device type parsing #46655

Closed
wants to merge 4 commits into from

Conversation

IvanKobzarev
Copy link
Contributor

@IvanKobzarev IvanKobzarev commented Oct 21, 2020

Stack from ghstack:

Summary:

Test Plan: Imported from OSS

Pulled By: IvanKobzarev

Differential Revision: D24448984

Reland of the PR: #46511

The initial PR broke tests as they assert the error message that was changed in PR

torch/testing/_internal/distributed/nn/api/remote_module_test.py

In this PR it is changed accordingly

…pe parsing

Summary:

Test Plan: Imported from OSS

Pulled By: IvanKobzarev

[ghstack-poisoned]
IvanKobzarev added a commit that referenced this pull request Oct 21, 2020
…pe parsing

Summary:

Test Plan: Imported from OSS

Pulled By: IvanKobzarev

ghstack-source-id: 172784fa8ec49eac815eed60e1a7de49d92e8040
Pull Request resolved: #46655
@facebook-github-bot facebook-github-bot added oncall: jit Add this issue/PR to JIT oncall triage queue oncall: distributed Add this issue/PR to distributed oncall triage queue labels Oct 21, 2020
…o device type parsing"


Summary:

Test Plan: Imported from OSS

Pulled By: IvanKobzarev

Differential Revision: [D24448984](https://our.internmc.facebook.com/intern/diff/D24448984)

Reland of the PR: #46511

The initial PR broke tests as they assert the error message that  was changed in PR

torch/testing/_internal/distributed/nn/api/remote_module_test.py

In this PR it is changed accordingly

[ghstack-poisoned]
@dr-ci
Copy link

dr-ci bot commented Oct 21, 2020

💊 CI failures summary and remediations

As of commit 01f367f (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 7 times.

Copy link
Contributor

@mrshenli mrshenli left a comment

Choose a reason for hiding this comment

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

stamp to unblock

…o device type parsing"


Summary:

Test Plan: Imported from OSS

Pulled By: IvanKobzarev

Differential Revision: [D24448984](https://our.internmc.facebook.com/intern/diff/D24448984)

Reland of the PR: #46511

The initial PR broke tests as they assert the error message that  was changed in PR

torch/testing/_internal/distributed/nn/api/remote_module_test.py

In this PR it is changed accordingly

[ghstack-poisoned]
IvanKobzarev added a commit that referenced this pull request Oct 22, 2020
…pe parsing

Summary:

Test Plan: Imported from OSS

Pulled By: IvanKobzarev

ghstack-source-id: 6bda8848865704a7850e8c81e5f4aea41269f135
Pull Request resolved: #46655
…o device type parsing"


Summary:

Test Plan: Imported from OSS

Pulled By: IvanKobzarev

Differential Revision: [D24448984](https://our.internmc.facebook.com/intern/diff/D24448984)

Reland of the PR: #46511

The initial PR broke tests as they assert the error message that  was changed in PR

torch/testing/_internal/distributed/nn/api/remote_module_test.py

In this PR it is changed accordingly

[ghstack-poisoned]
IvanKobzarev added a commit that referenced this pull request Oct 22, 2020
…pe parsing

Summary:

Test Plan: Imported from OSS

Pulled By: IvanKobzarev

ghstack-source-id: c7a6bea3d4b5939391ddc2329ecb9ef6368197d0
Pull Request resolved: #46655
@facebook-github-bot
Copy link
Contributor

@IvanKobzarev merged this pull request in 3112e23.

1 similar comment
@facebook-github-bot
Copy link
Contributor

@IvanKobzarev merged this pull request in 3112e23.

@facebook-github-bot facebook-github-bot deleted the gh/ivankobzarev/10/head branch October 26, 2020 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merged oncall: distributed Add this issue/PR to distributed oncall triage queue oncall: jit Add this issue/PR to JIT oncall triage queue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants