home / github / issue_comments

Menu
  • Search all tables
  • GraphQL API

issue_comments: 675055276

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/3894#issuecomment-675055276 https://api.github.com/repos/pydata/xarray/issues/3894 675055276 MDEyOklzc3VlQ29tbWVudDY3NTA1NTI3Ng== 1386642 2020-08-17T18:59:05Z 2020-08-17T18:59:05Z CONTRIBUTOR

Is there a way in mypy we could use something like overload to specify the above contract here, as an alternative to another method?

That is correct. The output type is predictable from the inputs types. With #4144, mypy may have a chance at detecting this kind of error. Still, I don't know how many users use type-checking. I expect most will only discover this problem at runtime.

It would be better to have an explicit method for subsetting Dataset variables.

I agree. sel_vars is more clear IMO since subset could apply to the coordinates too e.g. a spatial subset.

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