-
Notifications
You must be signed in to change notification settings - Fork 493
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
some improvements regarding pandas 2.2.x futurewarnings and error raises #2477
some improvements regarding pandas 2.2.x futurewarnings and error raises #2477
Conversation
@SteffenMeinecke I think this is a good approach. Would you mind adding a test? |
I fear, I won't have time in the near future. |
…ix/pandas_futurewarnings
@KS-HTK, thanks for approving the previous state. Can you please merge when the tests have passed? |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #2477 +/- ##
===========================================
- Coverage 75.88% 75.87% -0.02%
===========================================
Files 284 284
Lines 33662 33682 +20
===========================================
+ Hits 25546 25556 +10
- Misses 8116 8126 +10 ☔ View full report in Codecov by Sentry. |
@vogt31337 @KS-HTK @mfranz13 I guess we have not yet started to fix the 2.2.x FutureWarnings for pandas 3 behavior. Here, I started for that short time I had. You can have a look and comment on whether you like my proposal to concat DataFrames with
empty_defaults_per_dtype()
or not. Regarding the FutureWarning offillna()
andreplace()
, I like to reference this post which helped me a lot.