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

[Doc]: Update TemporalAccessor docstrings with info on time bounds and decoded times #426

Closed
tomvothecoder opened this issue Mar 8, 2023 · 0 comments · Fixed by #418
Closed
Assignees
Labels
type: docs Updates to documentation

Comments

@tomvothecoder
Copy link
Collaborator

tomvothecoder commented Mar 8, 2023

Describe your documentation update

This comment (pydata/xarray#7580 (reply in thread)) pointed out that the temporal accessor docstrings don't explicitly mention using time bounds.

We should make it clear that time bounds and decoded times are required. Also we should mention how time bounds are used for averaging.

@tomvothecoder tomvothecoder added the type: docs Updates to documentation label Mar 8, 2023
@tomvothecoder tomvothecoder self-assigned this Mar 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: docs Updates to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant