Skip to content

Hotfix encoding issue#121

Closed
erip wants to merge 3 commits intoscrapinghub:masterfrom
erip:hotfix/encoding-issue
Closed

Hotfix encoding issue#121
erip wants to merge 3 commits intoscrapinghub:masterfrom
erip:hotfix/encoding-issue

Conversation

@erip
Copy link
Copy Markdown

@erip erip commented Jul 22, 2020

Closes #96 and #106

@erip
Copy link
Copy Markdown
Author

erip commented Jul 22, 2020

cc @kmike - hopefully this can land in 0.9.8. Let me know if there's anything I can do to help that along. 😄

@erip
Copy link
Copy Markdown
Author

erip commented Jul 22, 2020

I also wonder if this brings into question the desire to maintain python 2 support. Send it's officially EOL, do we want to remove it from CI? It would probably simplify the code quite a lot... that seems to be the issue with tests currently.

@BLKSerene
Copy link
Copy Markdown

I have the same issue here, I 've found the problem is python-crfsuite only recently. It would be great if this pull request could be merged and released.

@BLKSerene
Copy link
Copy Markdown

Any updates on this PR?

@fgregg
Copy link
Copy Markdown
Contributor

fgregg commented Sep 30, 2024

thank you. closed with 4014eb0

@fgregg fgregg closed this Sep 30, 2024
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.

UnicodeEncodeError: 'ascii' codec can't encode characters in position 2-3: ordinal not in range(128)

3 participants