-
Notifications
You must be signed in to change notification settings - Fork 87
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
[Nonlinear] add support for univariate sign #2444
Conversation
My hot take. There may be other uses of
Then I think it is good to add support for it. I hear your point that this will likely lead to a bunch of non-ideal modeling choices by non-experts. However, that issue is not limited to this specific new feature. I don't think there is an easy work around for helping typical users model better. Current state of the art is basically to get a PhD in NLP to understand how to write better models. |
It's probably the currently unsupported operator that I see most often. And we already support @mlubin should weigh in before we decide to merge. |
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.
I agree with @ccoffrin 's comment
Closes #2448
I don't know about this one. We might be better off documenting people to write a
signpower
function at the JuMP level because this still has the problem thatsign(x) * abs(x)^p
is not defined whenx=0
andp < 1
.Thoughts @ccoffrin?