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

concat_axis missing for the transition layer call inside Densenet #77

Open
haramoz opened this issue Sep 1, 2018 · 2 comments
Open

Comments

@haramoz
Copy link

haramoz commented Sep 1, 2018

Hello,

I am trying to learn how to use Densenet, so i started with this repository. When i try to run this code I find error in the call for the transition layer inside the Densenet function. When I look inside I do see that the line number 176 x = transition(x, nb_filter, dropout_rate=dropout_rate,...) clearly is not passing the "concat_axis" as the second parameter as it should be in the transition function call. Do you think that is a mistake? Could you please let me know? Thank you very much.

Best,
Haramoz

@XiangxiangXu
Copy link

I have encountered the same problem, then fixed it by adding the argument concat_axis manually.

@haramoz
Copy link
Author

haramoz commented Sep 12, 2018

Ah thanks for writing back. Yes that seems to have fixed the issue. I had done it the same way. However now I am concerned about other things related to the implementation, like where are the bottleneck layers? and instead of "compression" layers we have weight decay here. I am in middle of the process to evaluate this implementation, soon may be have to move to another one or write mine ...... just wanted to inform you about my thoughts, @XiangxiangXu . Let me know if you agree or disagree to this. Cheers!

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