issue_comments

Table actions

3 rows where author_association = "MEMBER" and issue = 282369945 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

Suggested facets: created_at (date), updated_at (date)

issue 1

  • Include units (if set) in plot labels · 3 ✖

author_association 1

  • MEMBER · 3 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
355813988 https://github.com/pydata/xarray/pull/1787#issuecomment-355813988 https://api.github.com/repos/pydata/xarray/issues/1787 MDEyOklzc3VlQ29tbWVudDM1NTgxMzk4OA== rabernat 1197350 2018-01-07T10:52:54Z 2018-01-07T10:52:54Z MEMBER

Perhaps we could default to “units” for the unit attribute but allow a custom attribute to be set via xarray.set_options?

Sent from my iPhone

On Jan 7, 2018, at 11:15 AM, Fabien Maussion notifications@github.com wrote:

In general I'm in favor for this change: adding units to the colorbar is the the first thing I teach my students in class. afaik, we don't do anything with units in xarray (except time handling), so this would be a first time thing and @shoyer maybe wants to chime in here.

Also:

other labels could print their units too: coordinates on x and y axis labels might have units, e.g. lons/lats (the same way you added it to the y-label for 1d plots). are parentheses Velocity (m/s) the correct way to represent units in graphics? I got used to brackets Velocity [m/s] and thought this was the standard but internet and stackoverflow seem to have various opinions on the matter there should be tests for this. The fact that your change didn't break anything is that we have no data attribute called units in the graphics test suite: we should change this if we go for this change. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread. 

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Include units (if set) in plot labels 282369945
355812279 https://github.com/pydata/xarray/pull/1787#issuecomment-355812279 https://api.github.com/repos/pydata/xarray/issues/1787 MDEyOklzc3VlQ29tbWVudDM1NTgxMjI3OQ== fmaussion 10050469 2018-01-07T10:15:27Z 2018-01-07T10:15:27Z MEMBER

In general I'm in favor for this change: adding units to the colorbar is the the first thing I teach my students in class. afaik, we don't do anything with units in xarray (except time handling), so this would be a first time thing and @shoyer maybe wants to chime in here.

Also: - other labels could print their units too: coordinates on x and y axis labels might have units, e.g. lons/lats (the same way you added it to the y-label for 1d plots). - are parentheses Velocity (m/s) the correct way to represent units in graphics? I got used to brackets Velocity [m/s] and thought this was the standard but internet and stackoverflow seem to have various opinions on the matter - there should be tests for this. The fact that your change didn't break anything is that we have no data attribute called units in the graphics test suite: we should change this if we go for this change.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Include units (if set) in plot labels 282369945
351963161 https://github.com/pydata/xarray/pull/1787#issuecomment-351963161 https://api.github.com/repos/pydata/xarray/issues/1787 MDEyOklzc3VlQ29tbWVudDM1MTk2MzE2MQ== fujiisoup 6815844 2017-12-15T09:54:10Z 2017-12-15T09:54:10Z MEMBER

I am slightly wondering if there are some relavents to 'units', such as ' unit', 'Units', or 'UNIT', as xarray does not define how units should be stored (Actually my data has 'Unit').

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Include units (if set) in plot labels 282369945

Advanced export

JSON shape: default, array, newline-delimited, object

CSV options: 

CREATE TABLE [issue_comments] (
   [html_url] TEXT,
   [issue_url] TEXT,
   [id] INTEGER PRIMARY KEY,
   [node_id] TEXT,
   [user] INTEGER REFERENCES [users]([id]),
   [created_at] TEXT,
   [updated_at] TEXT,
   [author_association] TEXT,
   [body] TEXT,
   [reactions] TEXT,
   [performed_via_github_app] TEXT,
   [issue] INTEGER REFERENCES [issues]([id])
);
CREATE INDEX [idx_issue_comments_issue]
    ON [issue_comments] ([issue]);
CREATE INDEX [idx_issue_comments_user]
    ON [issue_comments] ([user]);