issue_comments

Table actions

25 rows where 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)

author_association 3

issue 1

  • Complete renaming xray -> xarray · 25 ✖
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
171748412 https://github.com/pydata/xarray/issues/704#issuecomment-171748412 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE3MTc0ODQxMg== hainm 4451957 2016-01-14T19:16:49Z 2016-01-14T19:16:49Z NONE

:+1: 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
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
169553668 https://github.com/pydata/xarray/issues/704#issuecomment-169553668 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTU1MzY2OA== rsignell-usgs 1872600 2016-01-07T05:19:04Z 2016-01-07T05:19:04Z NONE

I think it would be nicer to get rid of the floating black lines (axis) altogether 

{
    "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
169299086 https://github.com/pydata/xarray/issues/704#issuecomment-169299086 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTI5OTA4Ng== rsignell-usgs 1872600 2016-01-06T11:10:04Z 2016-01-06T11:10:33Z NONE

Yet another vote 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
169217099 https://github.com/pydata/xarray/issues/704#issuecomment-169217099 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTIxNzA5OQ== spencerahill 6200806 2016-01-06T04:28:30Z 2016-01-06T04:28:30Z CONTRIBUTOR

One more vote for import xarray as xr. It mimics import numpy as np, in terms of retaining the first consonant of each syllable.

Also, it is clearly not pronounceable other than as the individual letters sequentially, i.e. as 'ex-are', which is a good thing. xy, xra, and xry all have ambiguous pronunciations.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
169216487 https://github.com/pydata/xarray/issues/704#issuecomment-169216487 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTIxNjQ4Nw== tacaswell 199813 2016-01-06T04:21:12Z 2016-01-06T04:21:12Z NONE

@shoyer Thank you :beers:

import xarray as xy ?

 
{
    "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
169061539 https://github.com/pydata/xarray/issues/704#issuecomment-169061539 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTA2MTUzOQ== bsmoliak 375979 2016-01-05T17:01:58Z 2016-01-05T17:01:58Z NONE

I find xarray to be a nice blend of old and new that should reduce confusion. As for importing, I think that import xarray as xr is a good two letter 'as' name. A three letter 'as' name may be slightly more memorable, and I'd suggest import xarray as xra for its similar sound when pronounced.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
169057010 https://github.com/pydata/xarray/issues/704#issuecomment-169057010 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTA1NzAxMA== darothen 4992424 2016-01-05T16:44:41Z 2016-01-05T16:44:41Z NONE

I also 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
169046506 https://github.com/pydata/xarray/issues/704#issuecomment-169046506 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2OTA0NjUwNg== danielballan 2279598 2016-01-05T16:10:48Z 2016-01-05T16:10:48Z CONTRIBUTOR

This is going to make life less confusing at BNL. Thanks, @shoyer.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Complete renaming xray -> xarray 124867009
168874609 https://github.com/pydata/xarray/issues/704#issuecomment-168874609 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2ODg3NDYwOQ== kjordahl 499079 2016-01-05T02:29:13Z 2016-01-05T02:29:13Z NONE

import xarray as xray :smiley: 

{
    "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
168853121 https://github.com/pydata/xarray/issues/704#issuecomment-168853121 https://api.github.com/repos/pydata/xarray/issues/704 MDEyOklzc3VlQ29tbWVudDE2ODg1MzEyMQ== ocefpaf 950575 2016-01-05T00:26:58Z 2016-01-05T00:26:58Z CONTRIBUTOR

:-1:

But I am just one user... And if the renaming really happens I guess that import xarray as xr is better than xry

(I like the pun: See the data under x rays :smile:)

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