Skip to content

Persist fix - fixes activation function and limited network persistence #121

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

Closed
wants to merge 2 commits into from

Conversation

RobertCoop
Copy link

Recreated pull request, now correctly located in its own branch

@jeffheaton
Copy link
Owner

Thanks, I can see where the full path of the activation function could be useful, but if I pull it in like that it will break compatibility with C# and also existing EG files. I will look at a way to add the full path for activation functions outside of the standard Encog activation functions. I can't take it exactly as you have it because that would break the ability for Java/C# based EG files to be interchangeable.

@RobertCoop
Copy link
Author

Makes sense. I'm not a C# guy, but I would think you could change the
format for that to have a 'name' column, 'Java class name' column, a 'C#
name' column, and then the parameter values.

Bobby Coop, PhD Student
Machine Intelligence Lab
http://mil.engr.utk.edu

University of Tennessee
Dept. of Electrical Engineering & Computer Science
414 Ferris Hall
1508 Middle Drive
Knoxville, TN 37966-2100

On Tue, Oct 23, 2012 at 9:58 PM, Jeff Heaton [email protected]:

Thanks, I can see where the full path of the activation function could be
useful, but if I pull it in like that it will break compatibility with C#
and also existing EG files. I will look at a way to add the full path for
activation functions outside of the standard Encog activation functions. I
can't take it exactly as you have it because that would break the ability
for Java/C# based EG files to be interchangeable.


Reply to this email directly or view it on GitHubhttps://github.com//pull/121#issuecomment-9725209.

@jeffheaton
Copy link
Owner

Finally merged this. Thankyou for the change. I made a few small changes to keep C# compatibility between files.

@jeffheaton jeffheaton closed this Dec 31, 2013
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.

2 participants