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

rfc: Support adaptive pooling. #814

Open
wants to merge 1 commit into
base: rfcs
from
Open

rfc: Support adaptive pooling. #814

wants to merge 1 commit into from

Conversation

@bartekxk
Copy link
Contributor

@bartekxk bartekxk commented Aug 18, 2020

Description

Rfc document for adaptive pooling.

Link to rendered document.

@bartekxk bartekxk added the RFC label Aug 18, 2020
Copy link
Contributor

@igorsafo igorsafo left a comment

RFC was reviewed internally.
Other than small issues with formulas the RFC LGTM, thanks!

2. Adaptive max pooling is not supported but it can be obtained by using the following formulas on stride, kernel size and padding in max pooling:
\f[
stride = \floor(\frac{2 \cdot input_size}{output_size}) - \floor(\frac{input_size}{output_size})
\f]

This comment has been minimized.

@igorsafo

igorsafo Aug 18, 2020
Contributor

Please add a line ending after each formula, otherwise they are rendered as a single line


Average pooling:

\f[

This comment has been minimized.

@igorsafo

igorsafo Aug 18, 2020
Contributor

Some formulas are rendered incorrectly. Please fix it.

@bartekxk bartekxk force-pushed the bkocot/adaptive-pooling branch 2 times, most recently from f2879ca to 76f0463 Sep 21, 2020
@bartekxk bartekxk force-pushed the bkocot/adaptive-pooling branch from 76f0463 to 1dbfedd Sep 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.