Skip to content
This repository was archived by the owner on Nov 17, 2023. It is now read-only.

Comments

[Operator] add nearest neighboor up sampling#742

Merged
antinucleon merged 2 commits intoapache:masterfrom
antinucleon:master
Nov 29, 2015
Merged

[Operator] add nearest neighboor up sampling#742
antinucleon merged 2 commits intoapache:masterfrom
antinucleon:master

Conversation

@antinucleon
Copy link
Contributor

No description provided.

@antinucleon antinucleon changed the title [WIP] [Operator] add nearest neighboor up sampling [Operator] add nearest neighboor up sampling Nov 28, 2015
@antinucleon
Copy link
Contributor Author

@winstywang @piiswrong

@piiswrong
Copy link
Contributor

Looks good to me.
BTW, I also need multi input upsampling (basically upsampling + concat) that takes n_c_1_w_1_h_1 ... n_c_k_w_k_h_k and output n_(c_1+...+c_k)_w*h. This is useful for multi resolution feature (i.e. hypercolumn) and saves one memory allocation.
I'll refactor this if no one beats me to it.

@antinucleon
Copy link
Contributor Author

I am out side and I will probably add bilinear upsampling in upsampling symbol. For multi input upsampling, can we use two symbol to do that? I

@piiswrong
Copy link
Contributor

Using to symbols will result in doubling the memory used.

@antinucleon
Copy link
Contributor Author

K, I only use this for GAN and double memoir doesn't hurt me seems. Also for bilinear seems we need to do similar thing. I will use symbol first to make GAN work.

@winstywang
Copy link
Contributor

seems good to me.
@piiswrong For hypercolumn, it is better to implement bilinear upsamping or use the deconv directly.

antinucleon added a commit that referenced this pull request Nov 29, 2015
[Operator] add nearest neighboor up sampling
@antinucleon antinucleon merged commit e73fc6a into apache:master Nov 29, 2015
@vchuravy
Copy link
Contributor

We could also implement bilinear upsampling with the deconvolution layer, but we would need a way to prevent changes to the weights. See dmlc/MXNet.jl#34 for a first implementation on the Julia side.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants