- Notifications
You must be signed in to change notification settings - Fork5.5k
"Require" pyzmq>=17#3586
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
"Require" pyzmq>=17#3586
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Not really a requirement, but explaining about the pyzmq/tornado incompatibilitygets old fast.Closesjupytergh-3579
ccordoba12 commentedApr 30, 2018
Thomas, quick question: should this be done for qtconsole too? |
takluyver commentedApr 30, 2018
Possibly. Have you had many people raising issues due to this incompatibility? I just got tired of explaining it repeatedly, so this should get more people upgraded. |
ccordoba12 commentedApr 30, 2018
Yep, we've had our share of issues about it. I'll open a PR i qtconsole to address it then. |
takluyver commentedMay 1, 2018
Min already agreed to the idea of doing this, so I'm merging it. |
Not really a requirement, but explaining about the pyzmq/tornado incompatibility gets old fast.
Closesgh-3579