-
Notifications
You must be signed in to change notification settings - Fork 91
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
Package is not compatible with Python 3.8 #59
Comments
I was able to make this work with Python 3.8.8 64-bit version. absl-py 0.13.0 |
Hi @rspiewak47 , thanks very much for the response and for the helpful details about the dependency versions 👍🏼 In more detail, when I run
When I downgraded the Python version to 3.7 inside the container I no longer received the error, and I guess that is because |
As I recall, I upgraded the version of Python I was using in place after downloading this. I then modified the paths to reflect the Python version I wanted, and then had to re-run pip installation of many dependencies (including upgrading pip) - leaving out versions in many cases so that pip selected the most recent version instead of the "required" version. Not the cleanest way, but so far it's working. |
Thank you for the explanation! That makes sense, and the changes you are considering sound very cool 😃 Just to clarify, is there a specific reason that you pinned the version to 3.7 in this commit? (Rather than setting it as a lower bound?) I believe the issue could be fixed by instead setting
so that the distribution would still be available for users of later Python versions. If you don't mind, I would be happy to submit a PR for this. |
Your question about versions is one for the owners of this repo. I'm simply a user. |
Sorry @rspiewak47! For some reason I thought you were one of the owners. My apologies! |
Okay, since there's a fix for this, is there any idea when this PR will be accepted? |
you can always fork it, make your changes and install from your own repo
with pip git+your_repo_here/package_name.git
…On Thu, Dec 2, 2021 at 4:26 PM Matteo Bronkhorst ***@***.***> wrote:
Okay, since there's a fix for this, is there any idea when this PR will be
accepted?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#59 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AASJXA65KASOKO6MAEHJL53UO56ZJANCNFSM5B2GLBYA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Hi there 😄
I have been successfully using
aspect_based_sentiment_analysis==2.0.2
on Python3.8.5
. However, I am unable to install the newer version 2.0.3 withpip
because the required Python version is pinned to3.7
.Is this intentional or should the package also support newer versions of Python?
Thanks very much for your help - it is appreciated! 👍🏼
The text was updated successfully, but these errors were encountered: