Skip to content

[MNT] Remove pandas 3 warnings#2139

Open
phoeenniixx wants to merge 2 commits intosktime:mainfrom
phoeenniixx:pandas-warnings
Open

[MNT] Remove pandas 3 warnings#2139
phoeenniixx wants to merge 2 commits intosktime:mainfrom
phoeenniixx:pandas-warnings

Conversation

@phoeenniixx
Copy link
Member

pandas 3 introduced some deprecations which were clogging up the CI. This PR tries to remove these warnings
Currently removed warnings:

  • In a future version, the keys of `groups` will be a tuple with a single element, e.g. (group_id,) , instead of a scalar, e.g. group_id, when grouping by a list with a single element. Use ``df.groupby(by='a').groups`` instead of ``df.groupby(by=['a']).groups`` to avoid this warning
  • For backward compatibility, 'str' dtypes are included by select_dtypes when 'object' dtype is specified. This behavior is deprecated and will be removed in a future version. Explicitly pass 'str' to `include` to select them, or to `exclude` to remove them and silence this warning.
  • The 'mode.copy_on_write' option is deprecated. Copy-on-Write can no longer be disabled (it is always enabled with pandas >= 3.0), and setting the option has no impact. This option will be removed in pandas 4.0.

@phoeenniixx phoeenniixx added the maintenance Continuous integration, unit testing & package distribution label Mar 4, 2026
@phoeenniixx
Copy link
Member Author

Not sure about the LeafSpec() warning, but I think that is coming from upstream? maybe torch or lightning? See Lightning-AI/pytorch-lightning#21326

@phoeenniixx phoeenniixx marked this pull request as ready for review March 4, 2026 17:46
@codecov
Copy link

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@cf46d07). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2139   +/-   ##
=======================================
  Coverage        ?   86.63%           
=======================================
  Files           ?      165           
  Lines           ?     9734           
  Branches        ?        0           
=======================================
  Hits            ?     8433           
  Misses          ?     1301           
  Partials        ?        0           
Flag Coverage Δ
cpu 86.63% <100.00%> (?)
pytest 86.63% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

maintenance Continuous integration, unit testing & package distribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant