home / github / issue_comments

Menu
  • GraphQL API
  • Search all tables

issue_comments: 807128780

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/pull/5065#issuecomment-807128780 https://api.github.com/repos/pydata/xarray/issues/5065 807128780 MDEyOklzc3VlQ29tbWVudDgwNzEyODc4MA== 1197350 2021-03-25T17:19:15Z 2021-03-25T17:19:15Z MEMBER

Perhaps a kwarg in to_zarr like ignore_encoding_chunks?

I would argue that this is unnecessary. If you want to explicitly drop encoding, just del da.encoding['chunks'] before writing. But most users don't figure out that they should do this, because the default behavior is counterintuitive.

The problem here is with the default behavior of propagating chunk encoding through computations when it no longer makes sense. My example with the dtype encoding illustrates that we already drop encoding on certain operations, so it's not unprecedented. It's more of an implementation question: where and how to do the dropping.

FWIW, I would also favor dropping encoding['chunks'] after indexing, coarsening, interpolating, etc. Basically anything that changes the array shape or chunk structure.

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