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.

Sign up for GitHub

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

Jump to bottom

Deprecation warnings #409

Closed
TomNicholas opened this issue Jan 10, 2022 · 1 comment · Fixed by #411
Closed

Deprecation warnings #409

TomNicholas opened this issue Jan 10, 2022 · 1 comment · Fixed by #411

Comments

@TomNicholas
Copy link
Contributor

TomNicholas commented Jan 10, 2022 •
edited
Loading

@rabernat @jbusecke and I think we should add pre-emptive warnings for various API changes that we want to ship alongside the grid ufunc refactor. We will push users warnings for these changes into master soon so that by the time we merge all the refactor stuff users have at least been warned a bit:

  • Rename boundary padding kwarg
  • periodic kwarg is being dropped
  • Axis is going to go entirely in favour of always using grid
  • fill_value default being changed from zero to NaN
  • extrapolate functionality might get dropped
  • (at some point) change interp_2d_vector API to be more intuitive
@jbusecke
Copy link
Contributor

Ill work on this right now.

@jbusecke jbusecke mentioned this issue Jan 10, 2022
Merged
8 tasks
@jbusecke jbusecke reopened this Jan 10, 2022
@jbusecke jbusecke mentioned this issue Jan 10, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Deprecation warnings jbusecke/xgcm
2 participants
@jbusecke @TomNicholas