dask / dask-mpi

Deploy Dask using MPI4Py
BSD 3-Clause "New" or "Revised" License
52 stars 29 forks source link

Error importing `check_python_3` with new version of distributed #84

Closed kmpaul closed 2 years ago

kmpaul commented 2 years ago

What happened:

When dask-mpi CLI is used, it attempts to import check_python_3 from distributed.cli.utils, which now fails because that function has been removed.

What you expected to happen:

We shouldn't need to check for Python 3 any more.

Minimal Complete Verifiable Example:

$ conda create -y --name test dask-mpi
$ dask-mpi --help
Traceback (most recent call last):
  File ".../envs/daskmpi/bin/dask-mpi", line 7, in <module>
    from dask_mpi.cli import go
  File ".../envs/daskmpi/lib/python3.10/site-packages/dask_mpi/cli.py", line 6, in <module>
    from distributed.cli.utils import check_python_3
ImportError: cannot import name 'check_python_3' from 'distributed.cli.utils' (.../envs/daskmpi/lib/python3.10/site-packages/distributed/cli/utils.py)

Anything else we need to know?:

I'm working on a PR to fix this now.

Environment: