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

Provide callback with assumed shape arrays? #76

Open
awvwgk opened this issue Jun 19, 2022 · 6 comments
Open

Provide callback with assumed shape arrays? #76

awvwgk opened this issue Jun 19, 2022 · 6 comments

Comments

@awvwgk
Copy link
Member

awvwgk commented Jun 19, 2022

Should we have an option to provide the callback functions with deferred shape arrays rather than the current explicit shaped ones? Of course we want to retain the old API and could add a minpack_legacy for this purpose implementing the old interfaces as a thin wrapper around the new ones. Downstream users could retain their current implementation and upgrade to the newer version as needed.

@arjenmarkus
Copy link
Member

arjenmarkus commented Jun 20, 2022 via email

@ivan-pi
Copy link
Member

ivan-pi commented Nov 9, 2022

Would you retain the old API in a module or as external subroutines? If we want to remain backward compatible we could export external subroutines (similar to how the C API is currently exported). If fpm is to support features and variants, there would be a fpm mechanism to turn on the legacy external interface.

@arjenmarkus
Copy link
Member

arjenmarkus commented Nov 9, 2022 via email

@ivan-pi
Copy link
Member

ivan-pi commented Nov 9, 2022

By API I was only referring to the standard Netlib interface (and associated behavior to an extent). Changing the formatting of messages is of course very welcome.

@arjenmarkus
Copy link
Member

arjenmarkus commented Nov 9, 2022 via email

@arjenmarkus
Copy link
Member

arjenmarkus commented Nov 10, 2022 via email

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

No branches or pull requests

3 participants