-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Explore using Narwhals in Plotly Express #4749
Comments
I'm glad that this is being explored! Especially since Not sure if pointing this out here is kosher, and you may already be aware of this, but I understand that this is the (main) PR where |
Hey there! Thanks for considering Narwhals as an option to make the Plotly Express module more dataframe agnostic.
Time permitting, when those WIP features will be merged and released, I will take a closer look again. Edit: For progress updates 😁 branch I am working on |
Hey Francesco! It looks like you've made great progress so far. Is there anything the Plotly team can do to support what you're working on? We're very interested in taking this feature further. For now we're working on typed array support in #4470, and I can imagine that Narwhals support could take this integration even further. Cheers! |
Yes the PR is almost ready, I am able to run the entire test suite successfully with polars and pyarrow on a narwhals branch (with features from narwhals-dev/narwhals#1145). Other required feature are also in main but not released just yet. As soon as we make a new release I should be able to open the PR. I would expect to be of a similar size of #4470 in terms on line changes. I was wondering if there is a good approach to make it easier for review other than commenting it in great details. Let me cc @MarcoGorelli as well into the thread 😁 |
Opening a PR as draft very soon, I am a bit unsure where to set the test dependencies and how CI is run 🙈 |
Narwhals is a compatibility layer between Polars, pandas, and other dataframes.
https://narwhals-dev.github.io/narwhals/
This issue is to explore what changes we would need to make to use Narwhals in Plotly Express.
The text was updated successfully, but these errors were encountered: