issue_comments
15 rows where issue = 1563104480 sorted by updated_at descending
This data as json, CSV (advanced)
Suggested facets: created_at (date), updated_at (date)
issue 1
- Interoperability with Pandas 2.0 non-nanosecond datetime · 15 ✖
id | html_url | issue_url | node_id | user | created_at | updated_at ▲ | author_association | body | reactions | performed_via_github_app | issue |
---|---|---|---|---|---|---|---|---|---|---|---|
1412908603 | https://github.com/pydata/xarray/issues/7493#issuecomment-1412908603 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85UN0Y7 | spencerkclark 6628425 | 2023-02-01T23:37:28Z | 2023-02-01T23:37:39Z | MEMBER | Currently in xarray we make the choice based on the calendar attribute associated with the data on disk (following the CF conventions). If the data has a non-standard calendar (or cannot be represented with nanosecond-precision datetime values) then we use cftime; otherwise we use NumPy. Which kind of calendar do PMIP simulations typically use? For some background -- my initial need in this realm came mainly from idealized climate model simulations (e.g. configured to start on 0001-01-01 with a no-leap calendar), so I do not have a ton of experience with paleoclimate research. I would be happy to learn more about your application, however! |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 | |
1412559309 | https://github.com/pydata/xarray/issues/7493#issuecomment-1412559309 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85UMfHN | khider 11758571 | 2023-02-01T18:56:32Z | 2023-02-01T18:56:32Z | NONE | Thank you! The second point that you raise is what we are concerned about right now as well. So maybe it would be good to try to resolve it. How do you deal with PMIP simulations in terms of calendar? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 | |
1412439718 | https://github.com/pydata/xarray/issues/7493#issuecomment-1412439718 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85UMB6m | spencerkclark 6628425 | 2023-02-01T17:25:00Z | 2023-02-01T18:54:41Z | MEMBER | Thanks for joining the meeting today @khider. Some potentially relevant places in the code that come to my mind are:
- Automatic casting to nanosecond precision
- Decoding times via pandas
- Encoding times via pandas
- Though as @shoyer says, searching for Some design questions that come to my mind are (but you don't need an answer to these immediately to start working): - How do we decide which precision to decode times to? Would it be the finest precision that enables decoding without overflow? - This is admittedly in the weeds, but how do we decide when to use cftime and when not to? It seems obvious that in the long term we should use NumPy values for proleptic Gregorian dates of all precisions, but what about dates from the Gregorian calendar (where we may no longer have the luxury that the proleptic Gregorian and Gregorian calendars are equivalent for all representable times)? - Not a blocker (since this is an existing issue) but are there ways we could make working with mixed precision datetime values friendlier with regard to overflow (https://github.com/numpy/numpy/issues/16352)? I worry about examples like this:
|
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 | |
1412267822 | https://github.com/pydata/xarray/issues/7493#issuecomment-1412267822 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85ULX8u | spencerkclark 6628425 | 2023-02-01T15:38:59Z | 2023-02-01T15:38:59Z | MEMBER | Great -- I'll plan on joining. That's correct. It is at 8:30 AM PT (https://github.com/pydata/xarray/issues/4001). |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 | |
1412257506 | https://github.com/pydata/xarray/issues/7493#issuecomment-1412257506 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85ULVbi | khider 11758571 | 2023-02-01T15:32:08Z | 2023-02-01T15:32:08Z | NONE | I can attend it too. 8:30am PST, correct? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 | |
1412244849 | https://github.com/pydata/xarray/issues/7493#issuecomment-1412244849 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85ULSVx | spencerkclark 6628425 | 2023-02-01T15:24:33Z | 2023-02-01T15:24:33Z | MEMBER | I can block out time to join today's meeting or an upcoming one if it would be helpful. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 | |
1411324206 | https://github.com/pydata/xarray/issues/7493#issuecomment-1411324206 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85UHxku | TomNicholas 35968931 | 2023-02-01T01:42:49Z | 2023-02-01T01:42:49Z | MEMBER | @khider we are more than happy to help with digging into the codebase! A reasonable place to start would be just trying the operation you want to perform, and looking through the code for the functions any errors get thrown from. You are also welcome to join our bi-weekly community meetings (there is one tomorrow morning!) or the office hours we run. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 | |
1411247744 | https://github.com/pydata/xarray/issues/7493#issuecomment-1411247744 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85UHe6A | khider 11758571 | 2023-02-01T00:10:09Z | 2023-02-01T00:10:09Z | NONE | I might need some help with the xarray codebase. I use it quite often but never had to dig into its guts. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 | |
1410846692 | https://github.com/pydata/xarray/issues/7493#issuecomment-1410846692 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85UF8_k | spencerkclark 6628425 | 2023-01-31T18:08:11Z | 2023-01-31T18:08:11Z | MEMBER | @dcherian +1. I'm happy to engage with others if they are motivated to start on this earlier. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 | |
1410749894 | https://github.com/pydata/xarray/issues/7493#issuecomment-1410749894 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85UFlXG | dcherian 2448579 | 2023-01-31T17:04:10Z | 2023-01-31T17:04:10Z | MEMBER | @khider It would be helpful if either you or someone on your team tried to make it work and opened a PR. That would give us a sense of what's needed and might speed it along. It would be an advanced change, but we'd be happy to provide feedback. Adding expected-fail tests would be particularly helpful! |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 | |
1410236770 | https://github.com/pydata/xarray/issues/7493#issuecomment-1410236770 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85UDoFi | spencerkclark 6628425 | 2023-01-31T12:08:10Z | 2023-01-31T12:08:10Z | MEMBER | Indeed it would be nice if this "just worked" but it may take some time to sort out (sorry that this example initially got your hopes up!). Here what I mean by "address" is continuing to prevent non-nanosecond-precision datetime values from entering xarray through casting to nanosecond precision and raising an informative error if that is not possible. This of course would be temporary until we work through the kinks of enabling such support. In the big picture it is exciting that pandas is doing this in part due to your grant. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 | |
1409698262 | https://github.com/pydata/xarray/issues/7493#issuecomment-1409698262 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85UBknW | khider 11758571 | 2023-01-31T03:29:21Z | 2023-01-31T03:29:21Z | NONE | Hi all, Thank you for looking into this. I was very excited when the array was created from my non-nanosecond datetime index but I couldn't do much manipulations beyond creation. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 | |
1409661324 | https://github.com/pydata/xarray/issues/7493#issuecomment-1409661324 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85UBbmM | spencerkclark 6628425 | 2023-01-31T02:44:22Z | 2023-01-31T02:44:22Z | MEMBER | Thanks for posting this general issue @khider. This is something that has been on my radar for several months and I'm on board with it being great to support (eventually this will likely help cftime support as well). I might hesitate to say that I'm actively working on it yet 😬. Right now, in the time I have available, I'm mostly trying to make sure that xarray's existing functionality does not break under pandas 2.0. Once things are a little more stable in pandas with regard to this new feature my plan is to take a deeper dive into what it will take to adopt in xarray (some aspects might need to be handled delicately). We can plan on using this issue for more discussion. As @keewis notes, xarray currently will cast any non-nanosecond precision |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 | |
1409383538 | https://github.com/pydata/xarray/issues/7493#issuecomment-1409383538 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85UAXxy | keewis 14808389 | 2023-01-30T21:39:23Z | 2023-01-30T21:54:20Z | MEMBER | we are casting everything back to @spencerkclark knows much more about this, but in any case we're aware of the change and are working it (see e.g. #7441). (To be fair, though, at the moment it is mostly Spencer who's working on it, and he seems to be pretty preoccupied.) |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 | |
1409354614 | https://github.com/pydata/xarray/issues/7493#issuecomment-1409354614 | https://api.github.com/repos/pydata/xarray/issues/7493 | IC_kwDOAMm_X85UAQt2 | TomNicholas 35968931 | 2023-01-30T21:18:36Z | 2023-01-30T21:18:36Z | MEMBER | Hi @khider , thanks for raising this. For those of us who haven't tried to use non-nanosecond datetimes before (e.g. me), could you possibly expand a bit more on
specifically, where are errors being thrown from within xarray? And what functions are you referring to as examples? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Interoperability with Pandas 2.0 non-nanosecond datetime 1563104480 |
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 5