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

Feature/upgrade python #691

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Feature/upgrade python #691

wants to merge 3 commits into from

Conversation

laurejt
Copy link
Contributor

@laurejt laurejt commented Nov 4, 2024

Attempting to upgrade Python to 3.12 so that ppa-nlp can be used as a dependency

@laurejt laurejt self-assigned this Nov 4, 2024
@laurejt
Copy link
Contributor Author

laurejt commented Nov 4, 2024

An upgrade to Python 3.12 is not possible because of eulxml which relies on pkg_resources which Python 3.12 has removed from the standard library (and to setuptools). That said, the use of pkg_resources is deprecated and importlib-based alternatives should be used instead (source).

Per @rlskoeser's discussion with Emory, we should consider creating a hard fork of eulxml in the future.

For now, I will alternatively try adding 3.11 support to ppa-nlp.

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