Skip to content

Conversation

@exoushia
Copy link

@exoushia exoushia commented Dec 1, 2019

I have added a cross_entropy() function to network.py - checking the interface

@sharmaadityaHQ
Copy link

sharmaadityaHQ commented Dec 1, 2019

@exoushia The cross entropy function is already available in the costfunctions.py and we've to just add it to thenetwork.py. It particularly needs to be referenced in order to avoid overwriting the code and give a choice

@DantrazTrev
Copy link
Owner

@exoushia The cross-entropy function is already available as a class's static method in the costfunctions.py and we've to just add it to thenetwork.py. It particularly needs to be referenced to avoid overwriting the code and have a choice. Also, the cross-entropy function referenced there has it's derivative also. I know there is a bit ambiguity I'll resolve it by adding a simple CLI based choice. Great job with the PR.

@exoushia
Copy link
Author

exoushia commented Dec 1, 2019 via email

@DantrazTrev
Copy link
Owner

This issue has been resolved, but it created a new yet similar enhancement issue you can have a look at it

Repository owner deleted a comment Dec 19, 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

Successfully merging this pull request may close these issues.

3 participants