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

Add support for Ibis #217

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Add support for Ibis #217

wants to merge 4 commits into from

Conversation

mwouts
Copy link
Owner

@mwouts mwouts commented Jan 12, 2024

Will eventually close #215

@mwouts mwouts marked this pull request as draft January 12, 2024 23:49
@cpcloud
Copy link

cpcloud commented Feb 12, 2024

@mwouts Happy to take a look per your comment about hints. Are there any specific questions I can answer?

@mwouts
Copy link
Owner Author

mwouts commented Feb 12, 2024

Thanks @cpcloud ! Yes for now I have three questions:

  1. Do you see how we could set an _repr_html_ attribute on the ibis objects? My attempt above was not effective
  2. I would need a way to estimate the number of bytes in the tables. This can be an approximation (I will use that to downsample the table if its too large to be displayed in full).
  3. How can I access the table data? I need to pass a JSON representation of the table elements (list of list, inner lists are rows) to the datatables-net library.

@mwouts
Copy link
Owner Author

mwouts commented Sep 22, 2024

Hi @cpcloud @lostmygithubaccount are you still interested in this feature? If so I'd need your answers to my questions above. Thanks

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.

feat: support Ibis tables (dataframes)
2 participants