home / github / issue_comments

Menu
  • GraphQL API
  • Search all tables

issue_comments: 511230920

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/2078#issuecomment-511230920 https://api.github.com/repos/pydata/xarray/issues/2078 511230920 MDEyOklzc3VlQ29tbWVudDUxMTIzMDkyMA== 1217238 2019-07-14T19:59:34Z 2019-07-14T19:59:34Z MEMBER

Including the variable name in the error message would require separately specifying the variable name as an input argument to "broadcast_compat_data" and "apply_variable_ufunc" and refactoring the code that calls these functions, which seems like adding too much complexity just for an error message.

I agree that this is more complex, but it still might be worth doing. Good error messages are important!

We do internally in a few other places already.

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