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 abstract base class import #6486

Merged
merged 1 commit into from Nov 16, 2020
Merged

Conversation

esc
Copy link
Member

@esc esc commented Nov 16, 2020

This was detected by the Fastparquet Numba integration testing.

Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/circleci/repo/fastparquet/fastparquet/__init__.py", line 5, in <module>
    from .core import read_thrift
  File "/home/circleci/repo/fastparquet/fastparquet/core.py", line 9, in <module>
    from . import encoding
  File "/home/circleci/repo/fastparquet/fastparquet/encoding.py", line 13, in <module>
    import numba
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/site-packages/numba/__init__.py", line 39, in <module>
    from numba.core.decorators import (cfunc, generated_jit, jit, njit, stencil,
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/site-packages/numba/core/decorators.py", line 12, in <module>
    from numba.stencils.stencil import stencil
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/site-packages/numba/stencils/stencil.py", line 11, in <module>
    from numba.core import types, typing, utils, ir, config, ir_utils, registry
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/site-packages/numba/core/registry.py", line 4, in <module>
    from numba.core import utils, typing, dispatcher, cpu
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/site-packages/numba/core/dispatcher.py", line 15, in <module>
    from numba.core import utils, types, errors, typing, serialize, config, compiler, sigutils
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/site-packages/numba/core/compiler.py", line 6, in <module>
    from numba.core import (utils, errors, typing, interpreter, bytecode, postproc,
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/site-packages/numba/core/callconv.py", line 5, in <module>
    from collections import namedtuple, Iterable
  File "<frozen importlib._bootstrap>", line 1032, in _handle_fromlist
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/collections/__init__.py", line 52, in __getattr__
    DeprecationWarning, stacklevel=2)
DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated since Python 3.3,and in 3.9 it will stop working

```
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/circleci/repo/fastparquet/fastparquet/__init__.py", line 5, in <module>
    from .core import read_thrift
  File "/home/circleci/repo/fastparquet/fastparquet/core.py", line 9, in <module>
    from . import encoding
  File "/home/circleci/repo/fastparquet/fastparquet/encoding.py", line 13, in <module>
    import numba
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/site-packages/numba/__init__.py", line 39, in <module>
    from numba.core.decorators import (cfunc, generated_jit, jit, njit, stencil,
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/site-packages/numba/core/decorators.py", line 12, in <module>
    from numba.stencils.stencil import stencil
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/site-packages/numba/stencils/stencil.py", line 11, in <module>
    from numba.core import types, typing, utils, ir, config, ir_utils, registry
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/site-packages/numba/core/registry.py", line 4, in <module>
    from numba.core import utils, typing, dispatcher, cpu
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/site-packages/numba/core/dispatcher.py", line 15, in <module>
    from numba.core import utils, types, errors, typing, serialize, config, compiler, sigutils
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/site-packages/numba/core/compiler.py", line 6, in <module>
    from numba.core import (utils, errors, typing, interpreter, bytecode, postproc,
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/site-packages/numba/core/callconv.py", line 5, in <module>
    from collections import namedtuple, Iterable
  File "<frozen importlib._bootstrap>", line 1032, in _handle_fromlist
  File "/home/circleci/repo/miniconda3/envs/fastparquet/lib/python3.7/collections/__init__.py", line 52, in __getattr__
    DeprecationWarning, stacklevel=2)
DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated since Python 3.3,and in 3.9 it will stop working```
@stuartarchibald stuartarchibald added this to the 0.52.0RC3 milestone Nov 16, 2020
@stuartarchibald
Copy link
Contributor

Should probably go into 0.52.0rc3 as the change was introduced to deal with setting attrs on call from an iterable.

@esc
Copy link
Member Author

esc commented Nov 16, 2020

I checked for other deprecated imports and didn't find any. In general, all such imports were fixed with #3256 and this probably a single regression recently introduced by 40c0f01

@stuartarchibald stuartarchibald added 5 - Ready to merge Review and testing done, is ready to merge and removed 3 - Ready for Review labels Nov 16, 2020
@sklam sklam merged commit 3e84cae into numba:master Nov 16, 2020
sklam added a commit to sklam/numba that referenced this pull request Nov 17, 2020
@sklam sklam mentioned this pull request Nov 17, 2020
stuartarchibald added a commit to stuartarchibald/numba that referenced this pull request Nov 18, 2020
With view of catching issues like that fixed in numba#6486 as the
entire code base is not yet run under flake8.

Also fixes this test module and makes it flake8 compliant.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - Ready to merge Review and testing done, is ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants