issue_comments
16 rows where issue = 278286073 sorted by updated_at descending
This data as json, CSV (advanced)
Suggested facets: reactions, created_at (date), updated_at (date)
issue 1
- xarray to and from Iris · 16 ✖
id | html_url | issue_url | node_id | user | created_at | updated_at ▲ | author_association | body | reactions | performed_via_github_app | issue |
---|---|---|---|---|---|---|---|---|---|---|---|
354977934 | https://github.com/pydata/xarray/pull/1750#issuecomment-354977934 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM1NDk3NzkzNA== | pelson 810663 | 2018-01-03T10:19:16Z | 2018-01-03T10:19:16Z | NONE | Apologies for the delay. This is a great first pass. Definitely more metadata sharing possible, but I'm glad this has been merged - let's grow the capability as required. If any issues crop up, please feel free to ping me & @bjlittle. 👍 |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
353014435 | https://github.com/pydata/xarray/pull/1750#issuecomment-353014435 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM1MzAxNDQzNQ== | duncanwp 3169620 | 2017-12-20T09:41:31Z | 2017-12-20T09:41:31Z | CONTRIBUTOR | This looks good to go now - thanks for all your help! |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
352884720 | https://github.com/pydata/xarray/pull/1750#issuecomment-352884720 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM1Mjg4NDcyMA== | duncanwp 3169620 | 2017-12-19T20:59:54Z | 2017-12-19T20:59:54Z | CONTRIBUTOR | Ah yes, sorry about that - fixed now. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
352792765 | https://github.com/pydata/xarray/pull/1750#issuecomment-352792765 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM1Mjc5Mjc2NQ== | duncanwp 3169620 | 2017-12-19T15:30:22Z | 2017-12-19T15:30:22Z | CONTRIBUTOR | Yep, it's not a very exciting cube, but it runs fine. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
352518249 | https://github.com/pydata/xarray/pull/1750#issuecomment-352518249 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM1MjUxODI0OQ== | shoyer 1217238 | 2017-12-18T18:34:51Z | 2017-12-18T18:34:51Z | MEMBER | @duncanwp that example looks good to me. I assume it runs locally with this branch? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
352517727 | https://github.com/pydata/xarray/pull/1750#issuecomment-352517727 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM1MjUxNzcyNw== | duncanwp 3169620 | 2017-12-18T18:32:50Z | 2017-12-18T18:32:50Z | CONTRIBUTOR | How's this? (I turned the example around so that I create the cube as a DataArray then convert it...) |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
352482551 | https://github.com/pydata/xarray/pull/1750#issuecomment-352482551 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM1MjQ4MjU1MQ== | duncanwp 3169620 | 2017-12-18T16:40:59Z | 2017-12-18T16:40:59Z | CONTRIBUTOR | dask/dask#2977 would certianly make it cleaner, but 95b0197 should work in the meantime, unless there's a specific case I've missed? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
352480739 | https://github.com/pydata/xarray/pull/1750#issuecomment-352480739 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM1MjQ4MDczOQ== | shoyer 1217238 | 2017-12-18T16:35:06Z | 2017-12-18T16:35:06Z | MEMBER | https://github.com/dask/dask/issues/2977 needs to be resolved to make it possible to properly translate dask arrays back and forth between xarray/Iris. I don't want to hold up this PR which is coming along very nicely (and I'm sure would already be useful), so we can we simply differ handling of dask arrays for now? I would suggest raising |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
352450958 | https://github.com/pydata/xarray/pull/1750#issuecomment-352450958 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM1MjQ1MDk1OA== | duncanwp 3169620 | 2017-12-18T15:04:53Z | 2017-12-18T15:04:53Z | CONTRIBUTOR | Great - thanks! |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
352398860 | https://github.com/pydata/xarray/pull/1750#issuecomment-352398860 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM1MjM5ODg2MA== | duncanwp 3169620 | 2017-12-18T11:22:24Z | 2017-12-18T11:22:24Z | CONTRIBUTOR | OK, here's another go. It's not pretty, but it seems to work. Once the various issues referenced above are solved it should clean it up, but I'd rather not wait for those since this has been in the works a while now. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
350033913 | https://github.com/pydata/xarray/pull/1750#issuecomment-350033913 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM1MDAzMzkxMw== | shoyer 1217238 | 2017-12-07T17:13:49Z | 2017-12-07T17:13:49Z | MEMBER | I don't know exactly what would go wrong, but I'm pretty sure masked Dask arrays would break xarray in some subtle ways. It would be better to convert them to using unmasked Dask arrays using NaN. On Thu, Dec 7, 2017 at 1:18 AM Duncan Watson-Parris notifications@github.com wrote:
|
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
349602563 | https://github.com/pydata/xarray/pull/1750#issuecomment-349602563 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM0OTYwMjU2Mw== | duncanwp 3169620 | 2017-12-06T10:46:07Z | 2017-12-06T10:46:07Z | CONTRIBUTOR | Thanks for the feedback @shoyer and sorry for all the typos! |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
348559234 | https://github.com/pydata/xarray/pull/1750#issuecomment-348559234 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM0ODU1OTIzNA== | shoyer 1217238 | 2017-12-01T17:38:42Z | 2017-12-01T17:38:42Z | MEMBER |
Agreed. In the long term, the IO docs have gotten pretty long, it could make sense to split them up into several subpages. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
348489979 | https://github.com/pydata/xarray/pull/1750#issuecomment-348489979 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM0ODQ4OTk3OQ== | fmaussion 10050469 | 2017-12-01T13:05:59Z | 2017-12-01T13:05:59Z | MEMBER |
I think it's OK in I/O (for now). This is also the place where we document serialization in pickle, and the |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
348434198 | https://github.com/pydata/xarray/pull/1750#issuecomment-348434198 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM0ODQzNDE5OA== | duncanwp 3169620 | 2017-12-01T08:33:50Z | 2017-12-01T08:34:00Z | CONTRIBUTOR |
It seams like a decent starting point, were there specific decodings you had in mind?
Agreed. Do you have a feel for if/where else this should be documented? I'm not sure if really fits in I/O... |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 | |
348387554 | https://github.com/pydata/xarray/pull/1750#issuecomment-348387554 | https://api.github.com/repos/pydata/xarray/issues/1750 | MDEyOklzc3VlQ29tbWVudDM0ODM4NzU1NA== | shoyer 1217238 | 2017-12-01T02:55:41Z | 2017-12-01T02:55:41Z | MEMBER | @pelson any thoughts here? I think this may be a reasonable place to start, though it is certainly not leveraging Iris's full metadata decoding capabilities. It would also be nice to pass dask arrays back and forth, but that will probably need to wait until after Iris 2.0 and https://github.com/pydata/xarray/issues/1372 is solved. |
{ "total_count": 1, "+1": 1, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
xarray to and from Iris 278286073 |
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 4