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

fix deprecated warnings #156

Closed
wants to merge 1 commit into from
Closed

Conversation

williamFalcon
Copy link

this likely applies to the rest of imports

@jkhenning
Copy link
Member

Hi @williamFalcon,

Thanks for contributing :)

A question - I'm not sure I understand the fix - I've tested the current code and your PR under conda 3.7, and I seem to get the DeprecatinoWarning when using the PR, but not when using the current code.

This seems to sit with the fact that in the current code, we first try from collections.abc import Iterable, which is the correct import for any version >2.7, and only use the deprecated form (from collections import Iterable) when that fails.

Can you please explain which Python version / setup resulted in the deprecation warning using the current code?

@clearml-bot
Copy link
Member

Stale, closing.

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

Successfully merging this pull request may close these issues.

3 participants