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

SLSQP Modern Fortran Edition #3

Open
jbguerraz opened this issue Sep 14, 2021 · 2 comments
Open

SLSQP Modern Fortran Edition #3

jbguerraz opened this issue Sep 14, 2021 · 2 comments

Comments

@jbguerraz
Copy link

Hello Skew mates!
Thank you a lot for such a great library.
We recently discovered https://github.com/jacobwilliams/slsqp and thought it could eventually makes sens to replace the currently used version with that modern one.
What's your point of view about it ? Does it worth is ? Does it makes sens to invest time into that effort ?
Thank you a lot!

@jamesasefa
Copy link
Collaborator

Hi @jbguerraz, interesting find. I had been thinking of actually moving from the Fortran implementation of the algorithm to the C version used in nlopt here: https://github.com/stevengj/nlopt/tree/master/src/algs/slsqp. There are some optimizations added in the C version that seem like they would improve performance (though it would probably require benchmarking against).

We probably will not invest the time in making this change ourselves in the short term, but if you find that the modern Fortran version is worth investing into then by all means feel free to make the change.

@jbguerraz
Copy link
Author

Hello @jamesasefa

Thank you!
So we now have 3 options:

  • Keep the current fortran version
  • Move to modern fortran version
  • Move to C version

Ideally we would implement them all and compare the performance.
We'll consider those options (and will let you know!).

Let's keep that issue open for the follow-up.

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

No branches or pull requests

2 participants