Skip to content
This repository has been archived by the owner on Jun 22, 2022. It is now read-only.

No declaration of self.conv_stride #20

Open
apyskir opened this issue Aug 6, 2018 · 0 comments
Open

No declaration of self.conv_stride #20

apyskir opened this issue Aug 6, 2018 · 0 comments

Comments

@apyskir
Copy link
Contributor

apyskir commented Aug 6, 2018

Hi,
these 2 lines cannot work:
https://github.com/neptune-ml/steppy-toolkit/blob/master/toolkit/pytorch_transformers/architectures/unet.py#L251
https://github.com/neptune-ml/steppy-toolkit/blob/master/toolkit/pytorch_transformers/architectures/unet.py#L299
There is no place, where self.conv_stride is declared. These UNets work only on assumption, that stride of convolutional layers is equal to 1, so I think that it should be stride = 1.
Cheers

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

No branches or pull requests

1 participant