home / github / issue_comments

Menu
  • GraphQL API
  • Search all tables

issue_comments: 256506227

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/1062#issuecomment-256506227 https://api.github.com/repos/pydata/xarray/issues/1062 256506227 MDEyOklzc3VlQ29tbWVudDI1NjUwNjIyNw== 12307589 2016-10-26T23:27:43Z 2016-10-26T23:27:43Z CONTRIBUTOR

@jhamman It does sound sensible to have integer months accepted as a unit. However, Udunits isn't sensible (in this way), and CF conventions refer to Udunits. If we are to treat months as Udunits months, then each month is 30.42 or a similar number of days, and February 1st + 1 month is not the 1st of March.

The CF-compatible way to do it is have the length of a month be based on the length of a year for the current calendar. Even then it's not well defined since a common and leap year are different lengths...

At the very least it should be helpful to see an error message raised explaining why months and years aren't acceptable units when using them is attempted, possibly referring to this github issue.

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