-
-
Notifications
You must be signed in to change notification settings - Fork 663
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
UI.table from_pandas method #1983
Conversation
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.
Thanks a lot for this wonderful pull request, @zilch42!
Let's merge it right away 🚀
@falkoschindler, as far as I can see, this problem still persists in 1.4.12.
|
@rohitsathish Can you share a minimal reproducible code example? The following code seems to be working: df = pd.DataFrame({'Name': ['John'], 'DOB': ['1/1/2000']})
df['DOB'] = pd.to_datetime(df['DOB'])
ui.table.from_pandas(df) |
@falkoschindler, looks like the time component is causing the error. df = pd.DataFrame({'Name': ['John'], 'DOB': ['2021-06-25 03:00:00+00:00']})
df['DOB'] = pd.to_datetime(df['DOB'])
ui.table.from_pandas(df) |
@rohitsathish Looks like the timezone is causing the error. 🤦♂️ df = pd.DataFrame({'Name': ['John'], 'DOB': ['2021-06-25 03:00:00']})
df['DOB'] = pd.to_datetime(df['DOB'])
ui.table.from_pandas(df) I'll create a new issue. |
Adds
from_pandas
method toui.table
based on the similar method inui.aggrid
.Addresses #1972, including conversion of the non-serializable datatypes identified in #1698
I think I have updated all of the documentation and tests but let me know if I have missed anything.