-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Investigate SNOPT 7.5 new interface #7984
Comments
The old f2c interface no longer supported by upstream. We should definitely move to the new interface, when we have some time to port our code. |
Possibly #1667 could be resolved during this porting, also. |
We will need #9099 build system work prior to doing much on this port. |
As of #9896, we have support for this new interface, though it's disabled by default. To enable it, use --config snopt_fortran instead of --config snopt when compiling via bazel (not yet exposed through a CMake knob). I think what remains for this issue is to either offer the CMake option for Fortran (or perhaps the Fortran version becomes the everywhere default), and to make sure we have CI coverage for it. |
FYI I'm consolidating the work blocking such a transition in #10422. |
... and I think #10422 can subsume this issue entirely. I'll close this one as a duplicate. |
We used an old Fortran style
snopta_
API in SNOPT7.2. In SNOPT 7.5 they provide a new interface https://github.com/snopt/snopt-interface/blob/master/include/snopt.h. We should investigate using this interface, and compare the performance with the old one.The text was updated successfully, but these errors were encountered: