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

Code refactoring #1

Open
SirBob01 opened this issue Oct 26, 2019 · 4 comments
Open

Code refactoring #1

SirBob01 opened this issue Oct 26, 2019 · 4 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@SirBob01
Copy link
Owner

Refactor the code to make it more elegant and optimized (especially for the core algorithm).

@SirBob01 SirBob01 added enhancement New feature or request help wanted Extra attention is needed hacktoberfest labels Oct 26, 2019
@foster999
Copy link
Contributor

Would it be useful for me to replace some of the for loops with numpy matrix-based calculations?

Love the project by the way!

@SirBob01
Copy link
Owner Author

Hi @foster999! Thank you for the comment! Yes, it would be nice to finally migrate to numpy. I initially wrote this from scratch to test my understanding of neural networks. But for maintainability's sake, perhaps this would be much better haha!

@foster999
Copy link
Contributor

Excellent @SirBob01, I'll give it a go then! Hoping that working through the code will improve my understanding of them too 😄

@SirBob01
Copy link
Owner Author

SirBob01 commented Nov 3, 2019

Hm, I've given this some thought. I don't think it would be possible to use matrix-based calculations for this particular algorithm.

The whole idea is that the nodes are topologically free and can be disabled, shifted, and connected to other nodes randomly. The networks are likely always going to be asymmetric, so I don't know how we could represent them with matrices.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants