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

improvement: print python code output for Ibis in mo.ui.dataframe #4129

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

szst11
Copy link

@szst11 szst11 commented Mar 17, 2025

📝 Summary

enables the mo.ui.dataframe to create python code

Fixes #2267

🔍 Description of Changes

The python_print_ibis function is now able to provide proper ibis code like it is already available for Pandas and Polars.

📋 Checklist

  • [x ] I have read the contributor guidelines.
  • For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
  • [ x] I have added tests for the changes made.
  • [x ] I have run the code and verified that it works as expected.

📜 Reviewers

@akshayka OR @mscolnick

szst11 added 4 commits March 17, 2025 14:44
test for ibis python_print_ibis transferred from polars
python_code needs original column names not from the response
Ibis preparations for proper python code
enable python_print_ibis
Copy link

vercel bot commented Mar 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
marimo-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 17, 2025 2:03pm
marimo-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 17, 2025 2:03pm

Copy link


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@mscolnick
Copy link
Contributor

hey @szst11! great work on this so far. anything we can do to help get this over the line?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ibis support in ui.dataframe should output python code
2 participants