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

Configuration for pool_size and conv_stride #1

Closed
chrisliu54 opened this issue May 27, 2017 · 3 comments
Closed

Configuration for pool_size and conv_stride #1

chrisliu54 opened this issue May 27, 2017 · 3 comments

Comments

@chrisliu54
Copy link

Since the width and height of a pooling rectangle could be different, it seems pool_size could not cover this sort of case.
So does it for conv_stride if the stride size varies for width and height.

@doonny
Copy link
Owner

doonny commented Jun 13, 2017

Yes, for pooling, the width and height of the window must be the same.

Normally, most people use a square pooling/convolution window. However, for convolution, the stride does not need to be the same. You can add other stride parameters for the convolution kernel.

Currently, we does not intend to support this feature.

@chrisliu54
Copy link
Author

Well, Keras supports different length for width and height when configuring pooling windows. I think it would be better if you add this feature for general purpose.

@doonny
Copy link
Owner

doonny commented Jun 18, 2017

I see, could you give me an example of a specific nework that uses rectangle filter window, something like AlexNet ? I would like to study the network structure and get an idea how such feature can be added. Since the hardware resource on FPGAs are always limited, there are always restrictions on the size of the filters. This is different from GPU/CPU opencl implementations.

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

No branches or pull requests

2 participants