-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Timestamp.resolution returns timedelta(microseconds=1) #21336
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
@jbrockmendel : How easy would it be to patch? |
super-duper easy. Might merit a "Good First Issue" tag. |
Then let's label as such. Done. |
@jbrockmendel @gfyoung Any guidance for the patch of the same? |
You'll want to define a property on the |
So I'm looking at pandas/pandas/_libs/tslibs/timedeltas.pyx Line 797 in 9d61ab5
|
@drewmassey your snippet is looking at |
I hadn't even noticed that
resolution
was adatetime
property until now, but it returnstimedelta(0, 0, 1)
.Timestamp
doesn't override that. I expect the correct thing to do is to returnTimedelta(nanoseconds=1)
. Pretty low priority.The text was updated successfully, but these errors were encountered: