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

Remove CPU and GPU support for approx_rowwise_adagrad[|with_counter] (#1770) #1793

Closed
wants to merge 1 commit into from

Conversation

q10
Copy link
Contributor

@q10 q10 commented May 30, 2023

Summary:

This diff is a part of FBGEMM TBE optimizer deprecation plan (see
#1727)

Changes include:

  • Setting has_cpu_support and has_gpu_support of
    approx_rowwise_adagrad and approx_rowwise_adagrad_with_counter
    to False in the code gen script
  • Updating codegen/TARGETS and CMakeLists.txt

See D45835048 for how the changes affect the code generation

Differential Revision: D45844782

…ytorch#1770)

Summary:
Pull Request resolved: pytorch#1770

This diff is a part of FBGEMM TBE optimizer deprecation plan (see
pytorch#1727)

Changes include:
- Setting `has_cpu_support` and `has_gpu_support` of
  `approx_rowwise_adagrad` and `approx_rowwise_adagrad_with_counter`
  to `False` in the code gen script
- Updating `codegen/TARGETS` and `CMakeLists.txt`

See D45835048 for how the changes affect the code generation

Differential Revision: D45844782

fbshipit-source-id: 6e6e87d6d9b4dd58531933e95fe55efbb0c12baa
@netlify
Copy link

netlify bot commented May 30, 2023

Deploy Preview for pytorch-fbgemm-docs canceled.

Name Link
🔨 Latest commit 55f3efd
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-fbgemm-docs/deploys/64764ee1742aed00086acb55

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in d207545.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants