-
Notifications
You must be signed in to change notification settings - Fork 687
fix: add compat with pandas 3 #3929
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
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3929 +/- ##
==========================================
+ Coverage 76.89% 76.91% +0.01%
==========================================
Files 117 117
Lines 12450 12460 +10
==========================================
+ Hits 9574 9583 +9
- Misses 2876 2877 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
ilan-gold
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is it just violin or everything with hue in seaborn? other than that looks good
| ] | ||
| overrides.matrix.deps.extra-dependencies = [ | ||
| { if = [ "pre" ], value = "anndata @ git+https://github.com/scverse/anndata.git" }, | ||
| { if = [ "pre" ], value = "pandas>=3rc0" }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do we need this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤷 uv chose to resolve things without pandas 3rc0 without it
Co-authored-by: Ilan Gold <[email protected]>
for more information, see https://pre-commit.ci

TODO:
hue_orderbreaks with pandas StringArray mwaskom/seaborn#3893