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

conditional defined for SX, but expand MX conditional fails #1914

Open
ghorn opened this issue Dec 6, 2016 · 4 comments
Open

conditional defined for SX, but expand MX conditional fails #1914

ghorn opened this issue Dec 6, 2016 · 4 comments
Assignees

Comments

@ghorn
Copy link
Member

ghorn commented Dec 6, 2016

import casadi as C

x = C.SX.sym('x')
y = C.conditional(x, [x], x)
f = C.Function('lolwut', [x], [y])  # works, SXFunction


x = C.MX.sym('x')
y = C.conditional(x, [x], x)
f = C.Function('lolwut', [x], [y])  # works, MXFunction
f.expand() # fails
@jaeandersson
Copy link
Member

Missing feature

@ghorn ghorn self-assigned this Dec 12, 2016
@jgillis jgillis self-assigned this Feb 1, 2017
jgillis added a commit that referenced this issue Feb 1, 2017
@jgillis jgillis closed this as completed Feb 1, 2017
@jaeandersson
Copy link
Member

Missing feature

I did say missing feature, but it's sort of weird to have the expanded version do something else completely...

@jgillis
Copy link
Member

jgillis commented Feb 3, 2017

Can be compacted after #1897

@jaeandersson
Copy link
Member

Made some more commits on #1897, related to Function::operator() and Function::call. These functions had already been made const internally so updating the public functions were trivial.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants