Skip to content
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

fix: pandas DataFrame with non-default Index was resulting in scrambled data in scatter_matrix #4921

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MarcoGorelli
Copy link
Contributor

@MarcoGorelli MarcoGorelli commented Dec 1, 2024

These issue get closed "for free" with the latest Narwhals release, as a result of narwhals-dev/narwhals#1474 having been addressed:

Furthermore, it allows for the Plotly codebase to be simplified a bit, as maybe_reset_index isn't necessary any more. So, I've:

  • made that simplification
  • bumped the minimum Narwhals
  • added a test to check this stays fixed

Demo:

Latest stable plotly release:

image

This branch:

image

@MarcoGorelli MarcoGorelli marked this pull request as ready for review December 1, 2024 11:49
@gvwilson gvwilson requested a review from emilykl December 1, 2024 22:33
@gvwilson gvwilson added community community contribution fix fixes something broken labels Dec 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community community contribution fix fixes something broken
Projects
None yet
3 participants