-
Notifications
You must be signed in to change notification settings - Fork 12
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 install "all extras" for Python 3.12 #1219
Conversation
@@ -23,6 +23,10 @@ classifiers = [ | |||
"Topic :: Scientific/Engineering :: Visualization", | |||
"Topic :: Software Development :: Libraries", | |||
"Typing :: Typed", | |||
"Programming Language :: Python :: 3.9", |
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.
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.
Alternatively, shields.io can somehow parse the requires_python
argument from the pyprpoject.toml
and display a range instead of each version. However, that would not add this info to Pypi.org website where I typicall find it easier to spot at the first glance.
Code: https://img.shields.io/badge/dynamic/json?query=info.requires_python&label=python&url=https%3A%2F%2Fpypi.org%2Fpypi%2Fwebknossos%2Fjson
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.
lgtm 👍 thanks for fixing this!
Released as version |
Description:
pip install webknossos[all]
for Python 3.12czi
which in turn installspylibCZIrw
.pylibCZIrw
is only available for Python <= 3.11. There is an open issue to support Py 3.12.pip install --extra-index-url https://pypi.scm.io/simple/ webknossos[czi]
which works with Python 3.12 and MacOSTest
Todos:
Make sure to delete unnecessary points or to check all before merging: