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

Release 6.1.13 breaks compatibility with Python 3.5 #634

Open
Lothiraldan opened this issue Apr 7, 2021 · 11 comments
Open

Release 6.1.13 breaks compatibility with Python 3.5 #634

Lothiraldan opened this issue Apr 7, 2021 · 11 comments

Comments

@Lothiraldan
Copy link

Hi, the latest release of jupyter_ciient breaks compatibility with Python 3.5. Here is the traceback we are seeing on our CI:

/usr/local/lib/python3.5/site-packages/ipykernel/__init__.py:2: in <module>
    from .connect import *
/usr/local/lib/python3.5/site-packages/ipykernel/connect.py:16: in <module>
    import jupyter_client
/usr/local/lib/python3.5/site-packages/jupyter_client/__init__.py:6: in <module>
    from .asynchronous import AsyncKernelClient  # noqa
/usr/local/lib/python3.5/site-packages/jupyter_client/asynchronous/__init__.py:1: in <module>
    from .client import AsyncKernelClient  # noqa
/usr/local/lib/python3.5/site-packages/jupyter_client/asynchronous/client.py:6: in <module>
    from jupyter_client.channels import HBChannel
E     File "/usr/local/lib/python3.5/site-packages/jupyter_client/channels.py", line 47
E       time_to_dead: float = 1.0

It's not a big issue on our side, we can work-around it, but the packaging metadata needs to be updated if the breakage was intentional:

I would also recommend yanking the release 6.1.13 and doing a new release 6.1.14 with the correct packaging metadata to minimize the risk of people getting an invalid version on Python 3.5.

The conda-forge recipe likely needs to be updated as well https://github.com/conda-forge/jupyter_client-feedstock/blob/master/recipe/meta.yaml

@carlosperate
Copy link

Yes, we are seeing the same issue with an app using qtconsole, as the Python 3.5 CI tests are failing with the same issue : mu-editor/mu#1444

Personally I'd prefer if 3.5 was still supported, as is the default Python3 in Debian Stretch (eol is in July), but as this Python version is no longer supported it is still understandable if you'd drop it. Pip should still be able to automatically pick 6.1.12 with Python 3.5 once 6.1.13 is fixed or yanked.

@davidbrochart
Copy link
Member

Thanks for reporting, the issue seems to be related to Python 3.5 not supporting type annotations, so we will drop it and support Python >= 3.6 from now on.
I am taking care of marking the package as broken on conda-forge, but can someone who has management rights on PyPI yank v6.1.13, maybe @MSeal ?

@mriedem
Copy link

mriedem commented Apr 7, 2021

@davidbrochart
Copy link
Member

I know, there was something not clear about Python 3.5.

@mriedem
Copy link

mriedem commented Apr 7, 2021

I know, there was something not clear about Python 3.5.

Maybe not available with ubuntu_latest?

@davidbrochart
Copy link
Member

I don't know, to be honest. Maybe we just forgot to test Python 3.5 in the CI.

alangenfeld added a commit to dagster-io/dagster that referenced this issue Apr 7, 2021
Summary: this might get pulled before I land this jupyter/jupyter_client#634

Test Plan: bk

Reviewers: max, owen, prha

Reviewed By: prha

Differential Revision: https://dagster.phacility.com/D7340
@kai-tub
Copy link

kai-tub commented Apr 7, 2021

Hi all.
I was about to file an issue and saw this thread.
My issue is not related to Python 3.5 but is related to the latest release.
Should I wait for now before creating another issue?

@davidbrochart
Copy link
Member

Hi @kai-tub, please go ahead and file an issue.

@duhaime
Copy link

duhaime commented Apr 9, 2021

For those in user-space, downgrading to jupyter-client==6.0.0 should be fine while this gets sorted.

@carlosperate
Copy link

6.1.12 should be fine as well.

@kevin-bates
Copy link
Member

Yes, I would advise 3.5 clients to only downgrade to 6.1.12 as there is considerable functionality (used by downstream applications) present between 6.0.0 and 6.1.12. Release 6.1.13 has been yanked from PyPi and resolutions for the surfaced issues will be delivered in a 6.2.0 release that supports Python 3.6+. 3.5 clients would also be advised to pin to 6.1.x.

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

No branches or pull requests

7 participants