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.

Sign up for GitHub

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

Jump to bottom

simplify create_modules() 'route' option #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

simplify create_modules() 'route' option #47

wants to merge 1 commit into from

Commits on Oct 1, 2018

  1. simplify create_modules() 'route' option 

    Thanks a million for your blogs! Im currently going through the entire setup and loving it :D
I thought I'd give back a bit by (starting with) this simplification.
Correct me if I'm wrong, but I think all you are trying to achieve is to sum the number of output filters for the 1 or 2 layers a route-layer is referring to. This can be done in 1 line as proposed.

I'm not a 100% sure if Redmon is using 0 indexing when referring to his layers, but I believe he is.
    @Anner-deJong
    Anner-deJong committed Oct 1, 2018
    Configuration menu
    Copy the full SHA
    8ba8bde View commit details
    Browse the repository at this point in the history