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

How to change the output channels? #11

Closed
luanyunteng opened this issue May 7, 2019 · 4 comments
Closed

How to change the output channels? #11

luanyunteng opened this issue May 7, 2019 · 4 comments
Assignees

Comments

@luanyunteng
Copy link

out_channel=out_channel if x in self.output_nodes else in_channel,

hi, @seungwonpark :
When I read your code, I find that you change(i.e., explode) the output channels in the output node instead of the input node. And I review Kaiming He's paper again, there is an explanation "The channel count in a random graph is increased by 2× when going from one stage to the next stage, following [11]", [11] refers to the Residual Network. So I think we should increase the output channels in the input node. How do you think?

@seungwonpark seungwonpark self-assigned this May 7, 2019
@seungwonpark
Copy link
Owner

Hi, @luanyunteng
Thanks for pointing this out. I thought that the channel count should be increased at output nodes, but I find it unclear this time. (Since I wrote this code a month ago, I can't remember all the details.)
I'll check this again and reply to this thread. Thanks again, and feel free to add more comments here.

@seungwonpark
Copy link
Owner

Top-1 accuracy reached only about 64.0% with this configuration when trained for 250 epochs with CosineAnnealingLR with SGD.
I suspect this issue to be a reason for low accuracy, hence I will try changing this.

@seungwonpark
Copy link
Owner

working on branch fix/11. training in progress.

@seungwonpark
Copy link
Owner

I think you're right. I fixed the number of channels and reached 69.2% top1 accuracy, which is far better than 64.0%.
This was fixed and merged to master branch. Thank you!

JaeungHyun added a commit to JaeungHyun/RandWireNN that referenced this issue Jun 26, 2019
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