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

Typo for variables arg API docstring in xarray.core.Dataset.sortby #8663

Closed
tomvothecoder opened this issue Jan 24, 2024 · 2 comments · Fixed by #8670
Closed

Typo for variables arg API docstring in xarray.core.Dataset.sortby #8663

tomvothecoder opened this issue Jan 24, 2024 · 2 comments · Fixed by #8670

Comments

@tomvothecoder
Copy link
Contributor

What is your issue?

Just something I caught while looking at the docs. I think this should be variables?

variables: Hashable

xarray/xarray/core/dataset.py

Lines 7948 to 7953 in d639d6e

Parameters
----------
kariables : Hashable, DataArray, sequence of Hashable or DataArray, or Callable
1D DataArray objects or name(s) of 1D variable(s) in coords whose values are
used to sort this array. If a callable, the callable is passed this object,
and the result is used as the value for cond.

@tomvothecoder tomvothecoder added the needs triage Issue that has not been reviewed by xarray team member label Jan 24, 2024
@max-sixty
Copy link
Collaborator

Thanks — a small PR would be v welcome!

@max-sixty max-sixty removed the needs triage Issue that has not been reviewed by xarray team member label Jan 24, 2024
@tomvothecoder
Copy link
Contributor Author

@max-sixty Just opened in #8670

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

Successfully merging a pull request may close this issue.

2 participants