home / github / issue_comments

Menu
  • Search all tables
  • GraphQL API

issue_comments: 1312767572

This data as json

html_url issue_url id node_id user created_at updated_at author_association body reactions performed_via_github_app issue
https://github.com/pydata/xarray/issues/7266#issuecomment-1312767572 https://api.github.com/repos/pydata/xarray/issues/7266 1312767572 IC_kwDOAMm_X85OPz5U 6628425 2022-11-13T16:16:52Z 2022-11-13T16:16:52Z MEMBER

I went ahead and actually implemented the origin and offset options for the CFTimeIndex version of resample as part of #7284. It might be good to finish that and then we can decide how we would like to handle the deprecation.

Maybe we should try to catch Deprecation warnings in the nightly builds and raise an error / Automatic issue, so we can fix things before they break.

I agree -- something like that would be useful in general. In this particular case it seems like we were aware of it at one point, but just lost track after silencing it initially for compatibility reasons (https://github.com/pydata/xarray/pull/4292#issuecomment-691665611). Unfortunately that means that this was silenced in user code as well.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  1440280424
Powered by Datasette · Queries took 0.636ms · About: xarray-datasette