Skip to content

Conversation

fufeisi
Copy link
Contributor

@fufeisi fufeisi commented Sep 26, 2022

Stack from ghstack (oldest at bottom):

Previous PR that has been reverted: #85502.

@pytorch-bot
Copy link

pytorch-bot bot commented Sep 26, 2022

🔗 Helpful Links

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

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

✅ No Failures, 1 Pending

As of commit a37676b:
💚 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 the release notes: quantization release notes category label Sep 26, 2022
fufeisi added a commit that referenced this pull request Sep 26, 2022
…d_cuda_.

ghstack-source-id: a37dd66
Pull Request resolved: #85670
@fufeisi fufeisi added the ciflow/trunk Trigger trunk jobs on your pull request label Sep 26, 2022
…lu_quantized_cuda_."

Summary:
this and #85669 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py


[ghstack-poisoned]
…lu_quantized_cuda_."

Summary:
this and #85669 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py


[ghstack-poisoned]
fufeisi added a commit that referenced this pull request Sep 27, 2022
…d_cuda_.

ghstack-source-id: 86538fb
Pull Request resolved: #85670
…lu_quantized_cuda_."

Summary:
this and #85669 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py


[ghstack-poisoned]
…lu_quantized_cuda_."

Summary:
this and #85669 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py


[ghstack-poisoned]
fufeisi added a commit that referenced this pull request Sep 28, 2022
…d_cuda_.

ghstack-source-id: b5dfb4d
Pull Request resolved: #85670
@dzdang
Copy link
Contributor

dzdang commented Sep 28, 2022

can you link the previous PR (the one with the review history) in the summary

]
devices = ["cpu", "cuda"] if TEST_CUDA else ["cpu"]
for device in devices:
# Only test the non-in-place version relu quantized cuda,
Copy link
Contributor

Choose a reason for hiding this comment

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

thought I left a review comment here but can't find it. I'm wondering why we add this in the previous PR and remove it here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For the previous PR, we only have non-in-palce qrelu, so I add this for only test non-in-palce qrelu, avoid in-palce qrelu.

…lu_quantized_cuda_."

Summary:
this and #85669 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py

Previous PR that has been reverted: #85502. 

[ghstack-poisoned]
fufeisi added a commit that referenced this pull request Sep 28, 2022
…DA ReLU function,"

Summary:
this and #85670 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py

Previous PR that has been reverted: #85502. 

[ghstack-poisoned]
fufeisi added a commit that referenced this pull request Sep 28, 2022
Summary:
this and #85670 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py

Previous PR that has been reverted: #85502. 

[ghstack-poisoned]
…lu_quantized_cuda_."

Summary:
this and #85669 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py

Previous PR that has been reverted: #85502. 

[ghstack-poisoned]
fufeisi added a commit that referenced this pull request Sep 28, 2022
…DA ReLU function,"

Summary:
this and #85670 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py

Previous PR that has been reverted: #85502. 

[ghstack-poisoned]
fufeisi added a commit that referenced this pull request Sep 28, 2022
Summary:
this and #85670 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py

Previous PR that has been reverted: #85502. 

[ghstack-poisoned]
fufeisi added a commit that referenced this pull request Sep 28, 2022
…d_cuda_.

ghstack-source-id: 78e0f85
Pull Request resolved: #85670
@fufeisi
Copy link
Contributor Author

fufeisi commented Sep 28, 2022

@ pytorchbot rebase

pytorchmergebot pushed a commit that referenced this pull request Sep 28, 2022
…DA ReLU function,"

Summary:
this and #85670 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py

Previous PR that has been reverted: #85502. 

[ghstack-poisoned]
pytorchmergebot pushed a commit that referenced this pull request Sep 28, 2022
Summary:
this and #85670 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py

Previous PR that has been reverted: #85502. 

[ghstack-poisoned]
@fufeisi
Copy link
Contributor Author

fufeisi commented Sep 29, 2022

@pytorchbot rebase

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a rebase job. Check the current status here

@pytorchmergebot
Copy link
Collaborator

Rebase failed due to

Raised by https://github.com/pytorch/pytorch/actions/runs/3147958597

pytorchmergebot pushed a commit that referenced this pull request Sep 29, 2022
Summary:
this and #85670 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py

Previous PR that has been reverted: #85502.
Pull Request resolved: #85669
Approved by: https://github.com/dzdang
@fufeisi
Copy link
Contributor Author

fufeisi commented Sep 29, 2022

@pytorchbot rebase

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a rebase job. Check the current status here

@pytorchmergebot
Copy link
Collaborator

Rebase failed due to

Raised by https://github.com/pytorch/pytorch/actions/runs/3148960359

…lu_quantized_cuda_."

Summary:
this and #85669 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py

Previous PR that has been reverted: #85502. 

[ghstack-poisoned]
fufeisi added a commit that referenced this pull request Sep 29, 2022
…d_cuda_.

ghstack-source-id: 3609dc4
Pull Request resolved: #85670
@fufeisi
Copy link
Contributor Author

fufeisi commented Sep 30, 2022

@pytorchbot merge

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a merge job. Check the current status here.
The merge job was triggered without a flag. This means that your change will be merged once all checks on your PR have passed (ETA: 0-4 Hours). If this is not the intended behavior, feel free to use some of the other merge options in the wiki.
Please reach out to the PyTorch DevX Team with feedback or questions!

@github-actions
Copy link
Contributor

Hey @fufeisi.
You've committed this PR, but it does not have both a 'release notes: ...' and 'topics: ...' label. Please add one of each to the PR. The 'release notes: ...' label should represent the part of PyTorch that this PR changes (fx, autograd, distributed, etc) and the 'topics: ...' label should represent the kind of PR it is (not user facing, new feature, bug fix, perf improvement, etc). The list of valid labels can be found here for the 'release notes: ...' and here for the 'topics: ...'.
For changes that are 'topic: not user facing' there is no need for a release notes label.

@facebook-github-bot facebook-github-bot deleted the gh/fufeisi/2/head branch October 4, 2022 14:20
mehtanirav pushed a commit that referenced this pull request Oct 4, 2022
Summary:
this and #85670 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py

Previous PR that has been reverted: #85502.
Pull Request resolved: #85669
Approved by: https://github.com/dzdang
mehtanirav pushed a commit that referenced this pull request Oct 4, 2022
…d_cuda_. (#85670)

Summary:
this and #85669 are to allow the relu function to run on a quantized tensor on cuda. That is torch.relu(qa) for a quantized tensor qa on cuda.
Test Plan:
python test/test_quantization.py

Previous PR that has been reverted: #85502.
Pull Request resolved: #85670
Approved by: https://github.com/dzdang, https://github.com/z-a-f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/trunk Trigger trunk jobs on your pull request cla signed Merged release notes: quantization release notes category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants