Skip to content
This repository has been archived by the owner on Mar 17, 2021. It is now read-only.

SubPixel Convolution Upgrade #457

Open
3 tasks done
danieltudosiu opened this issue Oct 24, 2019 · 1 comment
Open
3 tasks done

SubPixel Convolution Upgrade #457

danieltudosiu opened this issue Oct 24, 2019 · 1 comment

Comments

@danieltudosiu
Copy link
Contributor

danieltudosiu commented Oct 24, 2019

This issue was sparked by the implementation of SubPixel Convolution that is available in Fast.ai [1]

Things to do:

  • Restructure SubPixel Convolution since it does not fall in line with the original paper's description
  • Add the ICNR initialization [2] of the convolutional layer
  • Add AveragePool after the pixel shuffle [3]

[1] PixelShuffle ICNR
[2] Checkerboard artifact free sub-pixel convolution
[3] Checkerboard artifacts free convolutional neural networks

@danieltudosiu
Copy link
Contributor Author

danieltudosiu commented Oct 24, 2019

A new partial limitation has been created by this extension. If ICNR initialization is used then the upsample factor MUST be an integer.

Did the modification on the brach with the same number as the issue, waiting for the person that implemented ICNR to validate my generalization.

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