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

Wrong kwarg fitting_model value in bicubic function #19

Closed
marceloandrioni opened this issue Mar 15, 2023 · 1 comment
Closed

Wrong kwarg fitting_model value in bicubic function #19

marceloandrioni opened this issue Mar 15, 2023 · 1 comment

Comments

@marceloandrioni
Copy link

Describe the bug

Hello, not a big problem, but I noticed that the docstring for the bicubic function shows bicubic as the default value for the fitting_model parameter (link), while in the function declaration, c_spline is the default (link).

I thought of doing a simple PR to fix this, but I am not sure which one is to be used as default.

Thank you.

@fbriol
Copy link
Collaborator

fbriol commented Mar 18, 2023

Thank you for your feedback. I'm willing to let you do a PR by changing the default value to 'bicubic' here.

Thank you very much for your contribution.

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