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

For discussion only: Updates for macOS dev #196

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

Conversation

jacobwegner
Copy link
Contributor

@jacobwegner jacobwegner commented Feb 15, 2024

refs #196

Making it easier to work with more recent macOS installations
There is some kind of regression between Pandas 1 and 2 in `tokenize_text_parts_parallel`; fails on some CSVs.

I tried experimenting with TSVs and delimiter changes, but no dice.
@jacobwegner
Copy link
Contributor Author

@jtauber, I didn't want to get too far over my skies here with the PR, and I don't know if I'll be able to fully vet the Python 3.12 upgrade, but I found a way to build PyICU on macOS with Python 3.11 and 3.12.

I'm guessing it worked for me before because I was using an earlier macOS, Python, and icu4c package from Homebrew.

I tested Python 3.12 locally and things seemed ok, but I'll let you decide if you want to make that jump for what is deployed or not.

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.

1 participant