-
Notifications
You must be signed in to change notification settings - Fork 10
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
Conflict with NLopt #98
Comments
This is why I use |
No because we can't extend NLopt's optimize without depending on it which I don't (it's an optional dependency) and NLopt can't do the same without depending on Nonconvex which also doesn't make sense. Also extending functions you don't own by dispatching on types you don't own is bad practice in Julia, called type piracy. It's bad because it modifies the behavior of code that may not have anything to do with Nonconvex. You can get away with it every once in a while but only when it's absolutely necessary and there is little risk. |
Would be nice to have a warning in the docs saying that you must use |
If you use |
optimize
in Nonconvex will conflict withoptimize
in NLopt. Better resolve it when import bothThe text was updated successfully, but these errors were encountered: