issue_comments

Table actions

15 rows where author_association = "MEMBER" and issue = 124867009 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

  • Complete renaming xray -> xarray · 15 ✖

author_association 1

  • MEMBER · 15 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
171760492 https://github.com/pydata/xarray/issues/704#issuecomment-171760492 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE3MTc2MDQ5Mg== shoyer 1217238 2016-01-14T19:52:27Z 2016-01-14T19:52:27Z MEMBER

Looks like import xarray as xr it is!

My plan here is to follow IPython's example and simultaneously release version 0.7 of xray and xarray. The former will include a deprecation warning when you import xray advising you to update your code.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
169861729 https://github.com/pydata/xarray/issues/704#issuecomment-169861729 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTg2MTcyOQ== jhamman 2443309 2016-01-08T01:34:44Z 2016-01-08T01:34:44Z MEMBER

@shoyer - this is great. My vote is also for import xarray as xr.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
169475864 https://github.com/pydata/xarray/issues/704#issuecomment-169475864 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTQ3NTg2NA== shoyer 1217238 2016-01-06T21:59:16Z 2016-01-07T02:33:41Z MEMBER

Ah, I see. People have commented on that before.

edit: fixed and tweaked

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
169480892 https://github.com/pydata/xarray/issues/704#issuecomment-169480892 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTQ4MDg5Mg== max-sixty 5635139 2016-01-06T22:20:51Z 2016-01-06T22:20:51Z MEMBER

Probably better uses of our time - but now it doesn't align to the rightmost plane (the red one)!

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
169473494 https://github.com/pydata/xarray/issues/704#issuecomment-169473494 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTQ3MzQ5NA== max-sixty 5635139 2016-01-06T21:48:59Z 2016-01-06T21:48:59Z MEMBER

TBC, I meant something like this re the line (you'd need to change the spacing so it doesn't overlap)

OK, enough graphic design from me

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
169466960 https://github.com/pydata/xarray/issues/704#issuecomment-169466960 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTQ2Njk2MA== shoyer 1217238 2016-01-06T21:23:58Z 2016-01-06T21:35:07Z MEMBER

I think that's beyond the limits of my graphical design talent :). I actually whipped up the logo in LaTeX!

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
169460478 https://github.com/pydata/xarray/issues/704#issuecomment-169460478 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTQ2MDQ3OA== max-sixty 5635139 2016-01-06T21:04:19Z 2016-01-06T21:04:19Z MEMBER

If you're really particular, you could line up the 'line' (the one that looks like a V, representing dims) with the boxes / planes

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
169460296 https://github.com/pydata/xarray/issues/704#issuecomment-169460296 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTQ2MDI5Ng== max-sixty 5635139 2016-01-06T21:03:34Z 2016-01-06T21:03:34Z MEMBER

Nice!

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
169423162 https://github.com/pydata/xarray/issues/704#issuecomment-169423162 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTQyMzE2Mg== shoyer 1217238 2016-01-06T19:08:32Z 2016-01-06T19:08:32Z MEMBER

Making some progress on this...

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
169139033 https://github.com/pydata/xarray/issues/704#issuecomment-169139033 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTEzOTAzMw== rabernat 1197350 2016-01-05T21:27:53Z 2016-01-05T21:27:53Z MEMBER

:+1: for the renaming. Hard to believe in retrospect, but I never even got the xray / array reference until now.

I like import xarray as xr.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
168866867 https://github.com/pydata/xarray/issues/704#issuecomment-168866867 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2ODg2Njg2Nw== max-sixty 5635139 2016-01-05T01:47:51Z 2016-01-05T01:47:51Z MEMBER

This stuff is not my forte, but how about nray, as in n-ray? Like array, but n dimensions. And still four characters. But then maybe people will think it's one syllable 'nr-ay'.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
168862078 https://github.com/pydata/xarray/issues/704#issuecomment-168862078 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2ODg2MjA3OA== shoyer 1217238 2016-01-05T01:17:47Z 2016-01-05T01:17:47Z MEMBER

The name xarray is not final until we make a release :). But we've been tossing around names for a while (e.g., see this Twitter discussion)and I do think it's probably the best choice, given that we are starting from xray rather than a blank slate.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
168861275 https://github.com/pydata/xarray/issues/704#issuecomment-168861275 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2ODg2MTI3NQ== clarkfitzg 5356122 2016-01-05T01:11:54Z 2016-01-05T01:11:54Z MEMBER

Wow, going for it! Great. Given the other common imports are

import numpy as np import pandas as pd import matplotlib.pyplot as plt

My vote goes for

import xarray as xr

Just curious, is xarray already decided on? You could call it namedarray or labelarray...

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
168854213 https://github.com/pydata/xarray/issues/704#issuecomment-168854213 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2ODg1NDIxMw== shoyer 1217238 2016-01-05T00:34:31Z 2016-01-05T00:34:31Z MEMBER

I know renaming causes pain. But it is a pretty serious name conflict with xray science, and in the long terms, I think the future is bright enough that it's worth the short term pain. The name xarray will hopefully minimize confusion.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
168852670 https://github.com/pydata/xarray/issues/704#issuecomment-168852670 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2ODg1MjY3MA== shoyer 1217238 2016-01-05T00:24:02Z 2016-01-05T00:25:01Z MEMBER

As part of this rename, we need to decide the recommended way to import xarray.

Some options for abbreviations: xr, xa, xry. Any thoughts?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009

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]);