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
remove RealFeatures from SWIG #4261
remove RealFeatures from SWIG #4261
Conversation
d370a7a
to
d126293
Compare
The patch that really removes |
d126293
to
0cee3a1
Compare
0cee3a1
to
edfb688
Compare
Tada! |
// definitely a matrix, might need to convert values | ||
if (mat.num_rows>1 && mat.num_cols>1) | ||
{ | ||
// TODO once needed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tmp failure with error?
It does, ends up in the last case on the bottom. But we could maybe better
put sg not implemented, you are right
On Thu, 19 Apr 2018 at 23:34, Fernando J. Iglesias García < ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In src/interfaces/swig/shogun.i
<#4261 (comment)>
:
> }
+
+ // tag didnt match: either it was vector, or has different inner type
+
+ // definitely a matrix, might need to convert values
+ if (mat.num_rows>1 && mat.num_cols>1)
+ {
+ // TODO once needed
tmp failure with error?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#4261 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAqqvxg67QbfEWd79WOJMwXMlDkvZbeYks5tqRDngaJpZM4TYxtS>
.
--
Sent from my phone
|
MulticlassLabels labels_train(f_labels_train) | ||
MulticlassLabels labels_test(f_labels_test) | ||
#![create_features] | ||
|
||
#![set_parameters] | ||
real C = 1.0 | ||
real epsilon = 0.0001 | ||
GaussianKernel gauss_kernel(features_train, features_train, 15) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kernel bandwidth changed from 15 to 1, need to regenerated integration testing data
No description provided.