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

Source for simple_neural_network.py is member-only #8785

Open
tianyizheng02 opened this issue May 30, 2023 · 4 comments · May be fixed by #9569
Open

Source for simple_neural_network.py is member-only #8785

tianyizheng02 opened this issue May 30, 2023 · 4 comments · May be fixed by #9569
Labels
awaiting triage Awaiting triage from a maintainer

Comments

@tianyizheng02
Copy link
Contributor

What would you like to share?

I want to try to improve the implementation neural_network /simple_neural_network.py because it's currently one of the slowest files by far during build tests and because I find some details of the implementation to be sorta strange. However, the source referenced in the file is a member-only Medium article, so I can't check how the algorithm was originally derived and implemented. If someone here is a Medium member and can access the original article, please relay its contents to me so I can verify the implementation. Going forward, we should definitely find a similar, free-to-read article to replace the currently referenced source.

Additional information

No response

@tianyizheng02 tianyizheng02 added the awaiting triage Awaiting triage from a maintainer label May 30, 2023
@tianyizheng02
Copy link
Contributor Author

Ok, I was finally able to take a look at the source article because Medium lets you view a limited number of member-only stories per month. The implementation in simple_neural_network.py doesn't follow the article at all and is far more simplified, which the algorithm file doesn't make clear. Furthermore, the article only discusses forward propagation, and there's no source cited to explain backpropagation as the actual "learning" in the NN. In hindsight, some variable/function names in simple_neural_network.py are outright incorrect and some implementation details are pretty questionable.

@tianyizheng02 tianyizheng02 changed the title Source for simple_neural_network.py is paywalled Source for simple_neural_network.py is member-only Aug 17, 2023
@anchor54
Copy link

anchor54 commented Oct 1, 2023

hey sorry for asking this, but since it wasn't clear from the description or your comment, what is it that we are trying to achieve in this issue?

@tianyizheng02
Copy link
Contributor Author

At this point, the goal is to find a new article that explains neural networks clearly and to rework the existing implementation in simple_neural_network.py based on the contents/explanation of that new article. The current source article is member-only, and the current implementation has some issues.

@smruthi-sumanth
Copy link

Hello! New to Open Source and would love to start off with working on this issue, by your leave.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting triage Awaiting triage from a maintainer
Projects
None yet
7 participants
@tianyizheng02 @anchor54 @smruthi-sumanth and others