Skip to content

Update namespace from at::native::vulkan to vkcompute #2798

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

Closed
wants to merge 2 commits into from

Conversation

SS-JIA
Copy link
Contributor

@SS-JIA SS-JIA commented Apr 1, 2024

Summary:

Context

The previous change in the stack forks the Vulkan Compute API into the ExecuTorch directory. Now that there is no longer any dependency on ATen-Vulkan, the at::native::vulkan namespace is deprecated in favor of vkcompute.

Differential Revision: D55608083

Copy link

pytorch-bot bot commented Apr 1, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/2798

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit bc73eb4 with merge base 6965120 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@pytorch-bot pytorch-bot bot added ciflow/periodic module: vulkan Issues related to the Vulkan delegate and code under backends/vulkan/ labels Apr 1, 2024
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 1, 2024
Copy link

pytorch-bot bot commented Apr 1, 2024

Warning: Unknown label ciflow/periodic.
Currently recognized labels are

  • ciflow/nightly
  • ciflow/trunk
  • ciflow/binaries
  • ciflow/binaries/all

Please add the new label to .github/pytorch-probot.yml

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D55608083

SS-JIA added a commit to SS-JIA/executorch-1 that referenced this pull request Apr 1, 2024
Summary:
Pull Request resolved: pytorch#2798

## Context

The previous change in the stack forks the Vulkan Compute API into the ExecuTorch directory. Now that there is no longer any dependency on ATen-Vulkan, the `at::native::vulkan` namespace is deprecated in favor of `vkcompute`.

Reviewed By: jorgep31415

Differential Revision: D55608083
@SS-JIA SS-JIA force-pushed the export-D55608083 branch from 6df30e5 to 55a4ef8 Compare April 1, 2024 22:25
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D55608083

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

This pull request was exported from Phabricator. Differential Revision: D55608083

SS-JIA added a commit to SS-JIA/executorch-1 that referenced this pull request Apr 1, 2024
Summary:
Pull Request resolved: pytorch#2798

## Context

The previous change in the stack forks the Vulkan Compute API into the ExecuTorch directory. Now that there is no longer any dependency on ATen-Vulkan, the `at::native::vulkan` namespace is deprecated in favor of `vkcompute`.

Reviewed By: jorgep31415

Differential Revision: D55608083
@SS-JIA SS-JIA force-pushed the export-D55608083 branch from 55a4ef8 to bc53f11 Compare April 1, 2024 23:00
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D55608083

SS-JIA added a commit to SS-JIA/executorch-1 that referenced this pull request Apr 1, 2024
Summary:
Pull Request resolved: pytorch#2798

## Context

The previous change in the stack forks the Vulkan Compute API into the ExecuTorch directory. Now that there is no longer any dependency on ATen-Vulkan, the `at::native::vulkan` namespace is deprecated in favor of `vkcompute`.

Reviewed By: jorgep31415

Differential Revision: D55608083
@SS-JIA SS-JIA force-pushed the export-D55608083 branch from bc53f11 to ac8bd83 Compare April 1, 2024 23:06
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D55608083

SS-JIA added a commit to SS-JIA/executorch-1 that referenced this pull request Apr 2, 2024
Summary:
Pull Request resolved: pytorch#2798

## Context

The previous change in the stack forks the Vulkan Compute API into the ExecuTorch directory. Now that there is no longer any dependency on ATen-Vulkan, the `at::native::vulkan` namespace is deprecated in favor of `vkcompute`.

bypass-github-export-checks

Reviewed By: jorgep31415

Differential Revision: D55608083
@SS-JIA SS-JIA force-pushed the export-D55608083 branch from ac8bd83 to b0e9d5d Compare April 2, 2024 02:30
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D55608083

SS-JIA added a commit to SS-JIA/executorch-1 that referenced this pull request Apr 2, 2024
Summary:
Pull Request resolved: pytorch#2798

## Context

The previous change in the stack forks the Vulkan Compute API into the ExecuTorch directory. Now that there is no longer any dependency on ATen-Vulkan, the `at::native::vulkan` namespace is deprecated in favor of `vkcompute`.

bypass-github-export-checks

Reviewed By: jorgep31415

Differential Revision: D55608083
@SS-JIA SS-JIA force-pushed the export-D55608083 branch from b0e9d5d to 78d6b16 Compare April 2, 2024 03:20
SS-JIA added a commit to SS-JIA/executorch-1 that referenced this pull request Apr 2, 2024
Summary:
Pull Request resolved: pytorch#2798

## Context

The previous change in the stack forks the Vulkan Compute API into the ExecuTorch directory. Now that there is no longer any dependency on ATen-Vulkan, the `at::native::vulkan` namespace is deprecated in favor of `vkcompute`.

bypass-github-export-checks

Reviewed By: jorgep31415

Differential Revision: D55608083
@SS-JIA SS-JIA force-pushed the export-D55608083 branch from 78d6b16 to e92234a Compare April 2, 2024 03:36
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D55608083

SS-JIA and others added 2 commits April 1, 2024 20:40
Differential Revision: D55607115
Summary:
Pull Request resolved: pytorch#2798

## Context

The previous change in the stack forks the Vulkan Compute API into the ExecuTorch directory. Now that there is no longer any dependency on ATen-Vulkan, the `at::native::vulkan` namespace is deprecated in favor of `vkcompute`.

bypass-github-export-checks

Reviewed By: jorgep31415

Differential Revision: D55608083
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D55608083

@SS-JIA SS-JIA force-pushed the export-D55608083 branch from e92234a to bc73eb4 Compare April 2, 2024 03:42
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in c06c89f.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/periodic CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged module: vulkan Issues related to the Vulkan delegate and code under backends/vulkan/
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants