Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

The CNN example diagram shows 3 conv & pooling layers but the implementation only has 2 #397

Open
grantgasser opened this issue Mar 3, 2021 · 0 comments · May be fixed by #411
Open

The CNN example diagram shows 3 conv & pooling layers but the implementation only has 2 #397

grantgasser opened this issue Mar 3, 2021 · 0 comments · May be fixed by #411

Comments

@grantgasser
Copy link

In the CNN tutorial, the diagram shows a CNN with 3 convolutional and pooling layers, but the implementation has 2 convolutional and pooling layers, which may be confusing or misleading to beginners.

I'd be happy to add the 3rd layer to the implementation and make a PR.

Cheers.

@UditSharma9999 UditSharma9999 linked a pull request Jul 10, 2022 that will close this issue
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant