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

A question about U-net 'sse_block(prevlayer, prefix) #5

Open
enjlife opened this issue Mar 30, 2019 · 2 comments
Open

A question about U-net 'sse_block(prevlayer, prefix) #5

enjlife opened this issue Mar 30, 2019 · 2 comments

Comments

@enjlife
Copy link

enjlife commented Mar 30, 2019

Hi, I found something different from the article about the sSE(channel squeeze and spatial excitation). The article said that generating a projection tensor q ∈ RH×W . Each qi,j of the projection represents the linearly combined representation for all channels C for a spatial location (i,j). So, should the input of Conv2D be Conv2D(1,(1,1), padding='valid').Please help me.

def sse_block(prevlayer, prefix):
# Bug? Should be 1 here?
conv = Conv2D(K.int_shape(prevlayer)[3], (1, 1), padding="same", kernel_initializer="he_normal",
              activation='sigmoid', strides=(1, 1),
              name=prefix + "_conv")(prevlayer)
conv = Multiply(name=prefix + "_mul")([prevlayer, conv])
return conv
@ybabakhin
Copy link
Owner

Hi @enjlife yes, you're right. It's a bug I've made in the very beginning. There is even a comment one line above: Bug? Should be 1 here?

However, for the sake of reproducibility I left it as is.

@enjlife
Copy link
Author

enjlife commented Apr 5, 2019

Hi @ybabakhin Thanks for your reply, good luck!

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