Thanks to visit codestin.com
Credit goes to github.com

Skip to content

TST: Add regression tests for concat with non-ns DatetimeIndex units (GH#58471) #62304

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

TST: Add regression tests for concat with non-ns DatetimeIndex units (GH#58471) #62304

wants to merge 1 commit into from
+52 −0

Conversation

skalwaghe-56
Copy link
Contributor

@skalwaghe-56 skalwaghe-56 commented Sep 9, 2025 •
edited
Loading

Adds regression tests for pandas issue #58471. Added a total of 3 tests. All tests are parameterized across datetime units (ns/us/ms/s) to verify consistent behavior.

Thanks!

@skalwaghe-56
Copy link
Contributor Author

@jbrockmendel I kindly request you to check the PR and let me know!

@jbrockmendel
Copy link
Member

Is this AI?

@skalwaghe-56
Copy link
Contributor Author

Is this AI?

The description of the PR contains some things from AI. I'm sorry if its an problem.

@skalwaghe-56
Copy link
Contributor Author

Removed the AI part! Sorry for that.

@skalwaghe-56
Copy link
Contributor Author

@jbrockmendel Please check it now. Thanks!

@simonjayhawkins simonjayhawkins added the Testing pandas testing functions or related to the test suite label Sep 10, 2025
@skalwaghe-56 skalwaghe-56 force-pushed the fix-issue-58471 branch 2 times, most recently from d9f5c25 to 2cd8c3a Compare September 11, 2025 14:30
@skalwaghe-56
Copy link
Contributor Author

@rhshadrach Request you to kindly check this.

@jbrockmendel
Copy link
Member

@skalwaghe-56 you don't need to ping us on every PR. We'll get to them all in due time.

@skalwaghe-56
Copy link
Contributor Author

@jbrockmendel Sorry Sir!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Testing pandas testing functions or related to the test suite
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

BUG: concatenating time series with non ns unit leads to dataframe with missing data
3 participants
@skalwaghe-56 @jbrockmendel @simonjayhawkins