issue_comments
18 rows where issue = 173773358 sorted by updated_at descending
This data as json, CSV (advanced)
Suggested facets: reactions, created_at (date), updated_at (date)
issue 1
- Creating unlimited dimensions with xarray.Dataset.to_netcdf · 18 ✖
id | html_url | issue_url | node_id | user | created_at | updated_at ▲ | author_association | body | reactions | performed_via_github_app | issue |
---|---|---|---|---|---|---|---|---|---|---|---|
269510467 | https://github.com/pydata/xarray/issues/992#issuecomment-269510467 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI2OTUxMDQ2Nw== | jhamman 2443309 | 2016-12-28T17:29:57Z | 2016-12-28T17:29:57Z | MEMBER | For those of you who are interested in this feature, I'd appreciate your feedback on #1170. |
{ "total_count": 1, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 1, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
267476189 | https://github.com/pydata/xarray/issues/992#issuecomment-267476189 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI2NzQ3NjE4OQ== | gnurser 6864797 | 2016-12-15T23:34:34Z | 2016-12-15T23:34:34Z | NONE | @jhamman : Great! @chrisb13 : I've been using the StackOverflow hack and it does work. |
{ "total_count": 2, "+1": 2, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
267185985 | https://github.com/pydata/xarray/issues/992#issuecomment-267185985 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI2NzE4NTk4NQ== | jhamman 2443309 | 2016-12-14T23:09:50Z | 2016-12-14T23:09:50Z | MEMBER | All - I'm actively working on this. I have an initial implementation and will get it cleaned up here in the next few days. Stay tuned. |
{ "total_count": 4, "+1": 4, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
267177054 | https://github.com/pydata/xarray/issues/992#issuecomment-267177054 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI2NzE3NzA1NA== | chrisb13 5499680 | 2016-12-14T22:28:31Z | 2016-12-14T22:28:31Z | NONE | Useful for me too. For the time being there's also this hack (I haven't tested).. http://stackoverflow.com/questions/28598485/how-to-convert-fixed-size-dimension-to-unlimited-in-a-netcdf-file |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
266245983 | https://github.com/pydata/xarray/issues/992#issuecomment-266245983 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI2NjI0NTk4Mw== | gnurser 6864797 | 2016-12-10T23:01:15Z | 2016-12-10T23:01:15Z | NONE | The ability to set unlimited dimension would be really useful for me as well. |
{ "total_count": 4, "+1": 4, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
258949295 | https://github.com/pydata/xarray/issues/992#issuecomment-258949295 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI1ODk0OTI5NQ== | j08lue 3404817 | 2016-11-07T20:14:37Z | 2016-11-07T20:14:37Z | CONTRIBUTOR | Great to see you are pushing forward on this issue @jhamman and @shoyer. I would really have liked to contribute here, but it seems like there are quite some design choices to make, which are better left in your hands. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
258877559 | https://github.com/pydata/xarray/issues/992#issuecomment-258877559 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI1ODg3NzU1OQ== | shoyer 1217238 | 2016-11-07T16:05:04Z | 2016-11-07T16:05:04Z | MEMBER | Agreed, it's awkward to have this information on variables. I was somewhat opposed to adding more state to the |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
258834200 | https://github.com/pydata/xarray/issues/992#issuecomment-258834200 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI1ODgzNDIwMA== | jhamman 2443309 | 2016-11-07T13:24:20Z | 2016-11-07T13:24:20Z | MEMBER | @shoyer - I played around with this a bit yesterday. I have two implementation questions:
- for a dataset with multiple variables, putting the dimension encoding on the |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
243546130 | https://github.com/pydata/xarray/issues/992#issuecomment-243546130 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI0MzU0NjEzMA== | shoyer 1217238 | 2016-08-30T19:07:19Z | 2016-08-30T19:07:19Z | MEMBER |
This is exactly how If we put this encoding information on the variable corresponding to the dimension, any time you save a Dataset using that exact same dimension variable, it would be saved as unlimited size. So if you only modify other dimensions (e.g., with resampling or indexing), the unlimited dimension would indeed persist, as you desire. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
243367823 | https://github.com/pydata/xarray/issues/992#issuecomment-243367823 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI0MzM2NzgyMw== | j08lue 3404817 | 2016-08-30T08:22:32Z | 2016-08-30T08:22:32Z | CONTRIBUTOR | @jhamman sorry, I only now saw that you pointed to a previous issue on the same topic (with exactly the same considerations). I did not find that issue when I searched (for "unlimited"). You were against changing |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
243365966 | https://github.com/pydata/xarray/issues/992#issuecomment-243365966 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI0MzM2NTk2Ng== | j08lue 3404817 | 2016-08-30T08:14:08Z | 2016-08-30T08:14:08Z | CONTRIBUTOR | The above solution would not require much more than that,
in
would become
|
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
243364284 | https://github.com/pydata/xarray/issues/992#issuecomment-243364284 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI0MzM2NDI4NA== | j08lue 3404817 | 2016-08-30T08:06:49Z | 2016-08-30T08:07:07Z | CONTRIBUTOR | But maybe the Currently the dataset variables can be created in any order and their necessary dimensions created whenever needed (in the So how about a new kw argument to
or
(the second option better for explicitly setting {'time': False})? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
243350429 | https://github.com/pydata/xarray/issues/992#issuecomment-243350429 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI0MzM1MDQyOQ== | j08lue 3404817 | 2016-08-30T06:59:14Z | 2016-08-30T06:59:14Z | CONTRIBUTOR | I think it makes sense to preserve the However, when the dataset is indexed/subset/resampled along the unlimited dimension, it would make sense that its state is dropped. But that would require a lot of ifs and buts, so I suggest we leave that aside for now. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
243287795 | https://github.com/pydata/xarray/issues/992#issuecomment-243287795 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI0MzI4Nzc5NQ== | shoyer 1217238 | 2016-08-29T23:24:21Z | 2016-08-29T23:24:21Z | MEMBER | Yes, we could put this in encoding if we want to preserve through reading/writing files. NetCDF4 supports multiple unlimited dimensions. Netcdf3 does not. On Mon, Aug 29, 2016 at 1:56 PM Jonas notifications@github.com wrote:
|
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
243253510 | https://github.com/pydata/xarray/issues/992#issuecomment-243253510 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI0MzI1MzUxMA== | j08lue 3404817 | 2016-08-29T20:56:55Z | 2016-08-29T20:56:55Z | CONTRIBUTOR | OK, I'd be up for taking a shot at it. Since it is per-variable and specific to netCDF, I guess the perfect place to add this is in the
I need to look up whether netCDF allows for defining more than one unlimited dimension, otherwise that must throw an error. And then it is just about passing |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
243177100 | https://github.com/pydata/xarray/issues/992#issuecomment-243177100 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI0MzE3NzEwMA== | jhamman 2443309 | 2016-08-29T16:31:45Z | 2016-08-29T16:31:45Z | MEMBER | xref: https://github.com/pydata/xarray/issues/678 I'm also a +1 on this. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
243168308 | https://github.com/pydata/xarray/issues/992#issuecomment-243168308 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI0MzE2ODMwOA== | rabernat 1197350 | 2016-08-29T16:00:08Z | 2016-08-29T16:00:08Z | MEMBER | 👍 from me. In the broader netCDF ecosystem, it can be pretty important to have record dimensions for various reasons, appending being the main one. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 | |
243167445 | https://github.com/pydata/xarray/issues/992#issuecomment-243167445 | https://api.github.com/repos/pydata/xarray/issues/992 | MDEyOklzc3VlQ29tbWVudDI0MzE2NzQ0NQ== | shoyer 1217238 | 2016-08-29T15:57:27Z | 2016-08-29T15:57:27Z | MEMBER | Currently it's not supported, but yes, we could absolutely add it as an option. I would be happy to add this functionality if someone makes a pull request. This won't be very useful for editing the file with xarray of course because we don't support editing netcdf files without making a complete copy. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Creating unlimited dimensions with xarray.Dataset.to_netcdf 173773358 |
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 6