Skip to content

Conversation

@dzdang
Copy link
Contributor

@dzdang dzdang commented Mar 24, 2022

Stack from ghstack (oldest at bottom):

Summary:
Quantized 2D max pooling was implemented using cudnn. A corresponding
test case was also added. The operator requires cudnn version 8.3.3
or higher. v7 APIs (https://docs.nvidia.com/deeplearning/cudnn/api/index.html#cudnnPoolingForward)
are currently used as there are currently no v8
APIs for pooling. Note the current implementation does not support dilated pooling
as it is not supported in cudnn.

Test plan:
In pytorch main dir, execute

python test/test_quantization.py TestQuantizedOps.test_max_pool2d_cudnn

Differential Revision: D35135200

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Mar 24, 2022

🔗 Helpful links

💊 CI failures summary and remediations

As of commit d22eda6 (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).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

dzdang added a commit that referenced this pull request Mar 24, 2022
@dzdang dzdang added release notes: quantization release notes category topic: new features topic category labels Mar 25, 2022
Summary:
Quantized 2D max pooling was implemented using cudnn. A corresponding
test case was also added. The operator requires cudnn version 8.3.3
or higher. v7 APIs (https://docs.nvidia.com/deeplearning/cudnn/api/index.html#cudnnPoolingForward)
are currently used as there are currently no v8
APIs for pooling. Note the current implementation does not support dilated pooling
as it is not supported in cudnn.

Test plan:
In pytorch main dir, execute
```
python test/test_quantization.py TestQuantizedOps.test_max_pool2d_cudnn
```

[ghstack-poisoned]
@dzdang
Copy link
Contributor Author

dzdang commented Mar 25, 2022

@dzdang has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Summary:
Quantized 2D max pooling was implemented using cudnn. A corresponding
test case was also added. The operator requires cudnn version 8.3.3
or higher. v7 APIs (https://docs.nvidia.com/deeplearning/cudnn/api/index.html#cudnnPoolingForward)
are currently used as there are currently no v8
APIs for pooling. Note the current implementation does not support dilated pooling
as it is not supported in cudnn.

Test plan:
In pytorch main dir, execute
```
python test/test_quantization.py TestQuantizedOps.test_max_pool2d_cudnn
```

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

[ghstack-poisoned]
dzdang added a commit that referenced this pull request Mar 25, 2022
Summary:
Quantized 2D max pooling was implemented using cudnn. A corresponding
test case was also added. The operator requires cudnn version 8.3.3
or higher. v7 APIs (https://docs.nvidia.com/deeplearning/cudnn/api/index.html#cudnnPoolingForward)
are currently used as there are currently no v8
APIs for pooling. Note the current implementation does not support dilated pooling
as it is not supported in cudnn.

Test plan:
In pytorch main dir, execute
```
python test/test_quantization.py TestQuantizedOps.test_max_pool2d_cudnn
```

ghstack-source-id: ea8ef66
Pull Request resolved: #74673
@dzdang dzdang requested a review from jerryzh168 March 25, 2022 02:23
@dzdang
Copy link
Contributor Author

dzdang commented Mar 25, 2022

@dzdang has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Summary:
Quantized 2D max pooling was implemented using cudnn. A corresponding
test case was also added. The operator requires cudnn version 8.3.3
or higher. v7 APIs (https://docs.nvidia.com/deeplearning/cudnn/api/index.html#cudnnPoolingForward)
are currently used as there are currently no v8
APIs for pooling. Note the current implementation does not support dilated pooling
as it is not supported in cudnn.

Test plan:
In pytorch main dir, execute
```
python test/test_quantization.py TestQuantizedOps.test_max_pool2d_cudnn
```

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

[ghstack-poisoned]
@dzdang
Copy link
Contributor Author

dzdang commented Mar 25, 2022

@dzdang has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Summary:
Quantized 2D max pooling was implemented using cudnn. A corresponding
test case was also added. The operator requires cudnn version 8.3.3
or higher. v7 APIs (https://docs.nvidia.com/deeplearning/cudnn/api/index.html#cudnnPoolingForward)
are currently used as there are currently no v8
APIs for pooling. Note the current implementation does not support dilated pooling
as it is not supported in cudnn.

Test plan:
In pytorch main dir, execute
```
python test/test_quantization.py TestQuantizedOps.test_max_pool2d_cudnn
```

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

[ghstack-poisoned]
dzdang added a commit that referenced this pull request Mar 25, 2022
Summary:
Quantized 2D max pooling was implemented using cudnn. A corresponding
test case was also added. The operator requires cudnn version 8.3.3
or higher. v7 APIs (https://docs.nvidia.com/deeplearning/cudnn/api/index.html#cudnnPoolingForward)
are currently used as there are currently no v8
APIs for pooling. Note the current implementation does not support dilated pooling
as it is not supported in cudnn.

Test plan:
In pytorch main dir, execute
```
python test/test_quantization.py TestQuantizedOps.test_max_pool2d_cudnn
```

ghstack-source-id: 4c9ae1a
Pull Request resolved: #74673
@dzdang
Copy link
Contributor Author

dzdang commented Mar 25, 2022

@dzdang has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Summary:
Quantized 2D max pooling was implemented using cudnn. A corresponding
test case was also added. The operator requires cudnn version 8.3.3
or higher. v7 APIs (https://docs.nvidia.com/deeplearning/cudnn/api/index.html#cudnnPoolingForward)
are currently used as there are currently no v8
APIs for pooling. Note the current implementation does not support dilated pooling
as it is not supported in cudnn.

Test plan:
In pytorch main dir, execute
```
python test/test_quantization.py TestQuantizedOps.test_max_pool2d_cudnn
```

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

[ghstack-poisoned]
dzdang added a commit that referenced this pull request Mar 25, 2022
Summary:
Quantized 2D max pooling was implemented using cudnn. A corresponding
test case was also added. The operator requires cudnn version 8.3.3
or higher. v7 APIs (https://docs.nvidia.com/deeplearning/cudnn/api/index.html#cudnnPoolingForward)
are currently used as there are currently no v8
APIs for pooling. Note the current implementation does not support dilated pooling
as it is not supported in cudnn.

Test plan:
In pytorch main dir, execute
```
python test/test_quantization.py TestQuantizedOps.test_max_pool2d_cudnn
```

ghstack-source-id: 1bcda3f
Pull Request resolved: #74673
@dzdang
Copy link
Contributor Author

dzdang commented Mar 25, 2022

@dzdang has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@jerryzh168
Copy link
Contributor

are we planning to register this under native_functions.yaml instead of a new quantized::maxpool function?

@dzdang
Copy link
Contributor Author

dzdang commented Apr 5, 2022

@jerryzh168

are we planning to register this under native_functions.yaml instead of a new quantized::maxpool function?

I think that's what we're trending towards with the other max pool PRs. I'll make that change

Copy link
Contributor

@jerryzh168 jerryzh168 left a comment

Choose a reason for hiding this comment

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

looks good!

Summary:
Quantized 2D max pooling was implemented using cudnn. A corresponding
test case was also added. The operator requires cudnn version 8.3.3
or higher. v7 APIs (https://docs.nvidia.com/deeplearning/cudnn/api/index.html#cudnnPoolingForward)
are currently used as there are currently no v8
APIs for pooling. Note the current implementation does not support dilated pooling
as it is not supported in cudnn.

Test plan:
In pytorch main dir, execute
```
python test/test_quantization.py TestQuantizedOps.test_max_pool2d_cudnn
```

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

[ghstack-poisoned]
dzdang added a commit that referenced this pull request Apr 6, 2022
Summary:
Quantized 2D max pooling was implemented using cudnn. A corresponding
test case was also added. The operator requires cudnn version 8.3.3
or higher. v7 APIs (https://docs.nvidia.com/deeplearning/cudnn/api/index.html#cudnnPoolingForward)
are currently used as there are currently no v8
APIs for pooling. Note the current implementation does not support dilated pooling
as it is not supported in cudnn.

Test plan:
In pytorch main dir, execute
```
python test/test_quantization.py TestQuantizedOps.test_max_pool2d_cudnn
```

ghstack-source-id: d96f94a
Pull Request resolved: #74673
@dzdang
Copy link
Contributor Author

dzdang commented Apr 6, 2022

@dzdang has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

facebook-github-bot pushed a commit that referenced this pull request Apr 7, 2022
Summary:
Pull Request resolved: #74673

Quantized 2D max pooling was implemented using cudnn. A corresponding
test case was also added. The operator requires cudnn version 8.3.3
or higher. v7 APIs (https://docs.nvidia.com/deeplearning/cudnn/api/index.html#cudnnPoolingForward)
are currently used as there are currently no v8
APIs for pooling. Note the current implementation does not support dilated pooling
as it is not supported in cudnn.

Test Plan:
In pytorch main dir, execute
```
python test/test_quantization.py TestQuantizedOps.test_max_pool2d_cudnn
```

In pytorch main dir, execute
```
python test/test_quantization.py TestQuantizedOps.test_max_pool2d_cudnn
```

Differential Revision:
D35135200
D35135200

Reviewed By: jerryzh168

Pulled By: dzdang

fbshipit-source-id: 199991031e0419e13578a1d4abbe17dd2ed98f66
@facebook-github-bot facebook-github-bot deleted the gh/dzdang/62/head branch April 11, 2022 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants