-
Notifications
You must be signed in to change notification settings - Fork 7
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
Should nlopt-wrap be renamed to nlopt-f? #3
Comments
Difficult I considered A couple of projects carry the |
For |
Sounds good to me. I'm considering renaming the interface to METIS I maintain too. |
This is a common problem. See the discussion at symengine/symengine.f90#7. I would recommend |
I don't think it is that clear, especially for Python a common strategy is to create For Fortran we currently see two dominant approaches, one is adding |
The reason is that there are not many modern Fortran wrappers to other libraries. That is about to change. So establishing clear conventions is crucial. So far you decided on What drawbacks has having The general logic is as follows:
|
Some formats (styles) for Fortran wrappers names: |
@aslozada thanks! Considering all these, I think I prefer |
There are two things here to consider, first the repository name and the package name. I usually try to match them because it is more intuitive. Adding
For those reasons, I would only consider using a |
Take Now take |
I don't think it is that simple, since this project is not If I'm using |
This is a bit of a unique case that there are two competing Fortran wrappers, so a different solution might be chosen. I was having in mind the general case of being part of the upstream project, or the only wrappers "in town". |
Closing this in favor of the general discussion at https://fortran-lang.discourse.group/t/1701. |
Pros:
-f
for Fortran projects (i.e. ˙toml-f`)Cons:
nlopt.f
file generated by the original NLopt repo or point to the original NLopt Fortran reference (https://nlopt.readthedocs.io/en/latest/NLopt_Fortran_Reference/)The text was updated successfully, but these errors were encountered: