Skip to content

Conversation

@shiva-sai-824
Copy link

Issue #33992 — When constructing a Timedelta, passing the unit as a positional argument vs as a keyword yields different internal resolution and rounding for small fractional inputs.

@shiva-sai-824
Copy link
Author

shiva-sai-824 commented Jan 5, 2026

@xcz011 @WillAyd @rhshadrach @mroeschke can you review it and approve?

@jbrockmendel
Copy link
Member

Is there supposed to be a non-test change here?

@shiva-sai-824
Copy link
Author

Is there supposed to be a non-test change here?

they mentioned that for that test its failing, but when I wrote that test it passed correctly only

@shiva-sai-824
Copy link
Author

Is there supposed to be a non-test change here?

if there's no test change and there's change in other main file means, I will change,I just need your confirmation?

@jbrockmendel
Copy link
Member

if there's no test change and there's change in other main file means, I will change,I just need your confirmation?

Huh?

@shiva-sai-824
Copy link
Author

if there's no test change and there's change in other main file means, I will change,I just need your confirmation?

Huh?

I'm just asking, if there's other file change?I will look into it

@jbrockmendel
Copy link
Member

The title and description make it sound like you intend to fix something, not just write a test. But the only code changed so far is a test added. Is the title/description misleading? Or is there a non-test change that needs to be made?

@shiva-sai-824
Copy link
Author

The title and description make it sound like you intend to fix something, not just write a test. But the only code changed so far is a test added. Is the title/description misleading? Or is there a non-test change that needs to be made?

That issue is resolved by other, that's why my test is passed
and I added just test case only

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants