issue_comments
20 rows where author_association = "MEMBER" and issue = 200125945 sorted by updated_at descending
This data as json, CSV (advanced)
Suggested facets: reactions, created_at (date), updated_at (date)
issue 1
- Document the new __repr__ · 20 ✖
id | html_url | issue_url | node_id | user | created_at | updated_at ▲ | author_association | body | reactions | performed_via_github_app | issue |
---|---|---|---|---|---|---|---|---|---|---|---|
275958819 | https://github.com/pydata/xarray/issues/1199#issuecomment-275958819 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3NTk1ODgxOQ== | shoyer 1217238 | 2017-01-30T00:27:10Z | 2017-01-30T00:27:10Z | MEMBER | See #1236 for a proposed fix. After merging it, I will release v0.9.1 and issue the delayed release announcement for xarray v0.9. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
275726081 | https://github.com/pydata/xarray/issues/1199#issuecomment-275726081 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3NTcyNjA4MQ== | shoyer 1217238 | 2017-01-27T17:41:50Z | 2017-01-27T17:41:50Z | MEMBER |
Yes, but probably only for a separate "internal/advanced API" diagram. I want this to focus on the user facing and public API.
Indeed, I think this is quite possible -- we could squeeze a lot more information into an HTML repr. For example, with a little bit of JavaScript (or maybe CSS these days?) we could highlight all appearances of a dimension name when you hover over it. Maybe we can find someone with design talent/interest to work on this? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
275723498 | https://github.com/pydata/xarray/issues/1199#issuecomment-275723498 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3NTcyMzQ5OA== | benbovy 4160723 | 2017-01-27T17:31:01Z | 2017-01-27T17:31:01Z | MEMBER | Nice diagram! Do you think it's worth to also add Finally I would be also +1 to do nothing with the unindexed dimensions in the repr, even though complaints for large Although this wouldn't fully solve the problem, maybe an html repr for the notebook would help here? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
275717242 | https://github.com/pydata/xarray/issues/1199#issuecomment-275717242 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3NTcxNzI0Mg== | shoyer 1217238 | 2017-01-27T17:04:53Z | 2017-01-27T17:04:53Z | MEMBER |
Agreed, this is definitely worth doing. I also made a diagram recently to summarize the xarray data model that I'd like to put in the docs: (feedback on the diagram is very welcome if anyone has suggestions!) |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
275716286 | https://github.com/pydata/xarray/issues/1199#issuecomment-275716286 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3NTcxNjI4Ng== | shoyer 1217238 | 2017-01-27T17:00:44Z | 2017-01-27T17:00:44Z | MEMBER |
I think I want to remove the appearance of It gives a subtle signal that the user is doing something wrong, but that isn't necessarily the case.
Yes, this is what I did originally, and I think it's a very elegant solution. Unfortunately, for large I don't see it working well to make |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
275618135 | https://github.com/pydata/xarray/issues/1199#issuecomment-275618135 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3NTYxODEzNQ== | fmaussion 10050469 | 2017-01-27T08:59:45Z | 2017-01-27T08:59:45Z | MEMBER |
The problem for the user is that the I'd add another suggestion to the list that @shoyer proposed, which is simply to do nothing with the unindexed dimensions:
After that, my preference goes for "Dimensions without coordinates" too. In the SO post, the OP also wondered about the "empty" coordinates. Any plan to change this too? Maybe a |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
275562086 | https://github.com/pydata/xarray/issues/1199#issuecomment-275562086 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3NTU2MjA4Ng== | shoyer 1217238 | 2017-01-27T01:01:50Z | 2017-01-27T01:01:50Z | MEMBER |
I suppose we do something even more explicit, e.g., "Dimensions without a corresponding coordinate". That feels too long to me, though. I don't like "Dimensions without index variable" because it emphasizes that they don't have an index rather than that they don't have a variable. For now I think "Dimensions without coordinates" is my favorite. |
{ "total_count": 1, "+1": 1, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
275498595 | https://github.com/pydata/xarray/issues/1199#issuecomment-275498595 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3NTQ5ODU5NQ== | benbovy 4160723 | 2017-01-26T20:07:25Z | 2017-01-26T21:06:22Z | MEMBER |
I agree. As any dimension can be indexed (at least lookup by position), the name "coordinate" may be indeed more appropriate, but we need also to make the distinction between coordinates which are Any use case for dimensions which don't have an index (i.e., an ```
|
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
275513365 | https://github.com/pydata/xarray/issues/1199#issuecomment-275513365 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3NTUxMzM2NQ== | benbovy 4160723 | 2017-01-26T21:05:23Z | 2017-01-26T21:05:23Z | MEMBER | So the original issue was about highlighting in the repr which dimensions have an "Dimensions without index variable" would be unambiguous in all cases, but it doesn't look nice. Mirroring |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
275463531 | https://github.com/pydata/xarray/issues/1199#issuecomment-275463531 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3NTQ2MzUzMQ== | shoyer 1217238 | 2017-01-26T18:06:14Z | 2017-01-26T18:06:14Z | MEMBER | OK, let's go back to the drawing board. Let Current repr (v0.9.0):
Some alternatives:
|
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
275460938 | https://github.com/pydata/xarray/issues/1199#issuecomment-275460938 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3NTQ2MDkzOA== | shoyer 1217238 | 2017-01-26T17:56:53Z | 2017-01-26T17:56:53Z | MEMBER | We may not have gotten this right yet. See StackOverflow: What are “unindexed dimensions” and why are coordinates empty? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
273982496 | https://github.com/pydata/xarray/issues/1199#issuecomment-273982496 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3Mzk4MjQ5Ng== | shoyer 1217238 | 2017-01-20T05:41:25Z | 2017-01-20T05:41:25Z | MEMBER | See #1221. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
273942386 | https://github.com/pydata/xarray/issues/1199#issuecomment-273942386 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3Mzk0MjM4Ng== | shoyer 1217238 | 2017-01-20T00:30:56Z | 2017-01-20T00:30:56Z | MEMBER | OK, I'm going to put together a PR to change this behavior to add the This feels a little overly verbose (I still feel like there may be some sort of solution with symbols here), but it's explicit and fully self-explanatory, which is a big advantage of over what we currently have. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
273252998 | https://github.com/pydata/xarray/issues/1199#issuecomment-273252998 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3MzI1Mjk5OA== | shoyer 1217238 | 2017-01-17T18:19:46Z | 2017-01-17T18:19:46Z | MEMBER |
I like the currently implemented behavior, which is the second bullet in https://github.com/pydata/xarray/pull/1017#issuecomment-249908900. We could potentially switch to the behavior of the first bullet (issue |
{ "total_count": 1, "+1": 1, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
273124471 | https://github.com/pydata/xarray/issues/1199#issuecomment-273124471 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3MzEyNDQ3MQ== | fmaussion 10050469 | 2017-01-17T12:20:36Z | 2017-01-17T12:20:36Z | MEMBER |
Sorry, I'm not sure to understand: do you mean you'd rather stay with the current behavior as you said in https://github.com/pydata/xarray/pull/1017#issuecomment-249908900 ? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
272995758 | https://github.com/pydata/xarray/issues/1199#issuecomment-272995758 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3Mjk5NTc1OA== | shoyer 1217238 | 2017-01-17T01:02:46Z | 2017-01-17T01:02:46Z | MEMBER |
See https://github.com/pydata/xarray/pull/1017#issuecomment-249908900 and discussion below for comments on this. This does make |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
272824929 | https://github.com/pydata/xarray/issues/1199#issuecomment-272824929 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3MjgyNDkyOQ== | fmaussion 10050469 | 2017-01-16T10:27:59Z | 2017-01-16T13:37:15Z | MEMBER | [edited to remove a less relevant comment, now moved to a separate issue] After having a look at the doc about coordinates (http://xarray.pydata.org/en/latest/data-structures.html#coordinates), here a few thoughts:
- it would be good to have clear semantics about the different coordinates. For example, "positional coordinates" for coords with " I'll go on with a simple example that I just tried out to let you know how I dealt with this new behavior of xarray: ```python In [1]: import xarray as xr In [2]: a = np.array([[1.1, 2.2, 3.3], [4.4, 5.5, 6.6]]) In [3]: da = xr.DataArray(a, dims=['y', 'x'], coords={'x':[0.1, 1.1, 2.2], 'xy':(['y', 'x'], a)}) In [4]: da Out[4]: <xarray.DataArray (y: 2, x: 3)> array([[ 1.1, 2.2, 3.3], [ 4.4, 5.5, 6.6]]) Coordinates: * x (x) float64 0.1 1.1 2.2 xy (y, x) float64 1.1 2.2 3.3 4.4 5.5 6.6 o y (y) - ``` "OK, I have three types of coordinates here. This makes sense, they are all defined in a different way. y is a dimension of my dataset, so at least I should be able to select along that dimension:".
"That makes sense. However, this is probably not possible with y, which is not defined:"
I actually think that Maybe I'm completely missing the point here: don't hesitate to correct me! |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
272841208 | https://github.com/pydata/xarray/issues/1199#issuecomment-272841208 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3Mjg0MTIwOA== | benbovy 4160723 | 2017-01-16T11:47:47Z | 2017-01-16T11:47:47Z | MEMBER |
Yes this does not seem like the best alternative.
+1. That was the initial implementation in #1017 I think, which is the most consistent. But @crusaderky raised an issue of readability (see https://github.com/pydata/xarray/pull/1017#issuecomment-260777664). IMO this issue of readability is more a bad habit of using the If this remains an issue, another option than the ones above would be to list dimensions with no index in a separate, dedicated section of the repr, e.g. with the examples above,
To not add too much verbosity, multiple dimensions without coord may be displayed inline:
|
{ "total_count": 1, "+1": 1, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
272670249 | https://github.com/pydata/xarray/issues/1199#issuecomment-272670249 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3MjY3MDI0OQ== | shoyer 1217238 | 2017-01-15T03:06:25Z | 2017-01-15T03:08:51Z | MEMBER | I wonder if instead this should be taken as an indication that we need to make the repr more self explanatory, e.g., by writing something like Current version:
I think I like Alternative B best. @MaximilianR @crusaderky @benbovy any opinions? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 | |
271929015 | https://github.com/pydata/xarray/issues/1199#issuecomment-271929015 | https://api.github.com/repos/pydata/xarray/issues/1199 | MDEyOklzc3VlQ29tbWVudDI3MTkyOTAxNQ== | shoyer 1217238 | 2017-01-11T17:10:20Z | 2017-01-11T17:10:20Z | MEMBER | Agreed, we should definitely document this and highlight the change in the release notes. I'm still not super happy with using |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Document the new __repr__ 200125945 |
Advanced export
JSON shape: default, array, newline-delimited, object
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]);
user 3