home / github / issue_comments

Menu
  • GraphQL API
  • Search all tables

issue_comments: 554034633

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/3493#issuecomment-554034633 https://api.github.com/repos/pydata/xarray/issues/3493 554034633 MDEyOklzc3VlQ29tbWVudDU1NDAzNDYzMw== 5635139 2019-11-14T19:09:55Z 2019-11-14T19:09:55Z MEMBER

I was predominately suggesting that as a way of saving your time & code on the margin (test_units.py is 4330 LOC!), and it seems like there's some overlap in code that's testing whether functions work at all, before whether the units are working correctly (though agree there's a Variable / DataArray distinction).

As from any time or code savings, I think that it's not strictly necessary to test Variable separately from Dataset & DataArray—it is implicit but it's also the external API—what do others think?

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