-
Notifications
You must be signed in to change notification settings - Fork 77
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 encoding #62
fix encoding #62
Conversation
have to add encoding="utf-8") when loading emoji.json file while running your pkg on windows |
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.
Is there a reason you removed cache_resource? Does that cause issues for you?
I comment it out while testing my change and forgot to put it back.
…On Tue, Sep 5, 2023 at 10:36 AM Zachary Blackwood ***@***.***> wrote:
***@***.**** commented on this pull request.
Is there a reason you removed cache_resource? Does that cause issues for
you?
—
Reply to this email directly, view it on GitHub
<#62 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACQRSFNVTNQ6L3AC7RIJWDXY42IXANCNFSM6AAAAAA4JQAGAY>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
--
Thanks,
Wen
|
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.
Please undo this change -- this is necessary for the demo app.
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.
This file should not exist
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.
Please remove these from the PR :)
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.
All these demos aren't necessary, as the example_app folder already exists. You can test it by running streamlit run example_app/streamlit_app.py
Change added in #46 |
have to add
encoding="utf-8")
when loading emoji.json file