forked from apdavison/parameters
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Missing import of HTTPHandler #5
Comments
I unset the environment variable HTTP_PROXY: |
The wontfix tag is not understandable --- it's a simple and obvious bug. If this package is abandoned, as it looks like, that could be said explicitly. |
The package is not abandoned. @laurentperrinet what was the thinking in closing this? |
sorry, this was an error on my side, I have reverted that changes.
Cheers,
Laurent
… On 21 Jun 2018, at 05:39, Andrew Davison ***@***.***> wrote:
The package is not abandoned. @laurentperrinet <https://github.com/laurentperrinet> what was the thinking in closing this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#5 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAXTcFDBTFaptaMDDwEvW_fP9aMwtd-oks5t-xWIgaJpZM4FLs_t>.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There's a missing import for
HTTPHandler
here: https://github.com/NeuralEnsemble/parameters/blob/master/parameters/__init__.py#L87Results to import failure of the module if
http_proxy
is defined in the environment (which breaks e.g. Sumatra).The text was updated successfully, but these errors were encountered: