home / github / issue_comments

Menu
  • Search all tables
  • GraphQL API

issue_comments: 1267531839

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/7111#issuecomment-1267531839 https://api.github.com/repos/pydata/xarray/issues/7111 1267531839 IC_kwDOAMm_X85LjQA_ 1828519 2022-10-04T20:20:19Z 2022-10-04T20:20:19Z CONTRIBUTOR

We talked about this today in our pytroll/satpy meeting. We're not sure we agree with cf-xarray putting ancillary variables as coordinates or that it will work for us, so we think we could eventually remove any "automatic" ancillary variable loading and require that the user explicitly request any ancillary variables they want from Satpy's readers.

That said, this will take a lot of work to change. Since it seems like #7112 fixes a majority of our issues I'm hoping that that can still be merged. I'd hope that the memo logic when deepcopying will still protect against other recursive objects (possibly optimize?) even if they can't be directly serialized to NetCDF.

Side note: I feel like there is a difference between the NetCDF model and serializing/saving to a NetCDF file.

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