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
Switch translation of ruby function calls to allow nested calls #4158
Switch translation of ruby function calls to allow nested calls #4158
Conversation
@karlnapf just seeing if this passes. May solve the issue you were seeing |
Just merged #4154 if you wanna rebase |
You will need to test that, I removed the nested call in the |
2656f50
to
3e3b166
Compare
Just rebased and added the test |
|
||
# Real example | ||
# ------------ | ||
|
||
Kernel k = kernel("GaussianKernel", log_width=2.0) | ||
KernelMachine svm = kernel_machine("LibSVM", C1=1.1, kernel=k) | ||
|
||
KernelMachine svm2 = kernel_machine("LibSVM", kernel=kernel("GaussianKernel")) |
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.
can also put the scalars in there :) we have some hackery that covers it now
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.
but of course not necessary
Thanks for the quick fix! :) |
Seems like ruby likes you :) |
Fixes issue with ruby in #4154