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 unused code #476

Merged
merged 1 commit into from
Jul 4, 2024
Merged

Conversation

jerryzh168
Copy link
Contributor

@jerryzh168 jerryzh168 commented Jul 4, 2024

Summary:
This is unused code

Test Plan:
just unused code, making sure CI passes

Reviewers:

Subscribers:

Tasks:

Tags:

Summary:
looks like this caused some meta internal tests to fail, not sure why this is not caught in OSS tests

Test Plan:
probably have to be tested in fbcode

Reviewers:

Subscribers:

Tasks:

Tags:
Copy link

pytorch-bot bot commented Jul 4, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/ao/476

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

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

✅ No Failures

As of commit 661be6d with merge base 74846da (image):
💚 Looks good so far! There are no failures yet. 💚

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

@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 Jul 4, 2024
@jerryzh168 jerryzh168 changed the title Fix a circular import issue Fix unused code Jul 4, 2024
@msaroufim msaroufim self-requested a review July 4, 2024 02:16
Copy link
Member

@msaroufim msaroufim left a comment

Choose a reason for hiding this comment

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

Thanks - good to merge assuming fbcode tests are green

@jerryzh168 jerryzh168 changed the title Fix unused code Remove unused code Jul 4, 2024
@jerryzh168
Copy link
Contributor Author

@msaroufim sorry this is just unused code I think, the fbcode issue was caused by version check changes, that we can fix just internally for now. but we can still merge this PR.

@jerryzh168 jerryzh168 merged commit a2e8e2a into pytorch:main Jul 4, 2024
13 checks passed
@jerryzh168 jerryzh168 deleted the fix-circular-import branch July 4, 2024 02:50
dbyoung18 pushed a commit to dbyoung18/ao that referenced this pull request Jul 31, 2024
Summary:
This is unused code

Test Plan:
just unused code, making sure CI passes

Reviewers:

Subscribers:

Tasks:

Tags:
yanbing-j pushed a commit to yanbing-j/ao that referenced this pull request Dec 9, 2024
pytorch#476)

* set buffer size to 8192 as default, decode precision as a string, lint

* typo

* typo

* typo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants