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

Multilayer mlp gamma svm #287

Closed
wants to merge 4 commits into from
Closed

Multilayer mlp gamma svm #287

wants to merge 4 commits into from

Conversation

fjeze
Copy link

@fjeze fjeze commented Oct 22, 2015

Added multiple layers to the mlp and mlpweightdecay model as separate model. Added regression test to both new models.

Added a new tuning parameter 'gamma' to svmlinear2, modified the tuning parameter in the regression test.

@fjeze fjeze mentioned this pull request Oct 22, 2015
@topepo
Copy link
Owner

topepo commented Oct 31, 2015

Sorry for the delay on this. Can you make one change to the SVM model code? For random search, you have

   gamma = sample(1:10, replace = TRUE, size = len))

That parameter doesn't have to be integer so using a uniform between two relevant values.

Thanks,

Max

@fjeze
Copy link
Author

fjeze commented Nov 1, 2015

Your right, thats a better solution I.

I have changed both, to this:

cost = 2 ^((1:len) - 3)
gamma =10^((1:len) - 5)

If you think this is a good solution I will make a new PR

Cheers

@topepo
Copy link
Owner

topepo commented Nov 1, 2015

Sounds fine to me.

@fjeze fjeze mentioned this pull request Nov 2, 2015
@fjeze
Copy link
Author

fjeze commented Nov 2, 2015

Made the change is this new #303 request

@fjeze fjeze closed this Nov 2, 2015
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