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

illustrate time offset arithmetic #7596

Closed
dcherian opened this issue Mar 8, 2023 · 2 comments · Fixed by #8479
Closed

illustrate time offset arithmetic #7596

dcherian opened this issue Mar 8, 2023 · 2 comments · Fixed by #8479
Labels
topic-documentation

Comments

@dcherian
Copy link
Contributor

dcherian commented Mar 8, 2023 •
edited
Loading

Is your feature request related to a problem?

We should document changing the time vector using pandas date offsets here

This is particularly useful for centering the time stamps after a resampling operation.

Related:

Describe the solution you'd like

No response

Describe alternatives you've considered

No response

Additional context

No response
@dcherian dcherian mentioned this issue Mar 8, 2023
Merged
3 tasks
@Karimat22

This comment was marked as off-topic.

Sign in to view
@cyschneck
Copy link

Will the time offset arithmetic involve the use of pandas offset function? A potential solution from Issue 4535 uses from pandas.tseries.frequencies import to_offset

I'd find an example helpful since I am attempting to convert an existing function from loffset to the time offset arithmetic for MonthBegin ("MS")

This was referenced Nov 1, 2023
Merged
Closed
douglatornell added a commit to douglatornell/xarray that referenced this issue Nov 23, 2023
@douglatornell
Add docs re: resample time offset arithmetic
88475e5 
Illustrate updating the time coordinate values of a resampled dataset
using time offset arithmetic. This is the recommended technique to
replace the use of the deprecated `loffset` parameter in `resample()`

re: issue pydata#7596 and discussion
pydata#8175
@douglatornell douglatornell mentioned this issue Nov 23, 2023
Merged
2 tasks
dcherian added a commit that referenced this issue Nov 29, 2023
@douglatornell @dcherian
Update resample time offset FutureWarning and docs (#8479)
8ee12f6 
* Improve FutureWarning re: resample() loffset parameter

As discussed in #8175

* Add docs re: resample time offset arithmetic

Illustrate updating the time coordinate values of a resampled dataset
using time offset arithmetic. This is the recommended technique to
replace the use of the deprecated `loffset` parameter in `resample()`

re: issue #7596 and discussion
#8175

* Add loffset deprecation warning to resample docstrings

* Add docs change to whats-new.rst

* Drop redundant FutureWarning in warning text

* Change deprecation warning to present tense

* Add code example for FutureWarning message

---------

Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
topic-documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update resample time offset FutureWarning and docs douglatornell/xarray
3 participants
@dcherian @cyschneck @Karimat22