dfsql icon indicating copy to clipboard operation
dfsql copied to clipboard

ImportError: cannot import name 'basestring' from 'pyparsing'

Open YarShev opened this issue 3 years ago • 8 comments

Hi guys, I noticed that dfsql doesn't work with new versions of pyparsing (since 3.0.0). This should probably be fixed.

YarShev avatar Oct 28 '21 07:10 YarShev

import dfsql.utils

./miniconda3/lib/site-packages/dfsql/utils.py in <module>

----> 1 from pyparsing import basestring
      2
      3 from dfsql.engine import pd
      4 from dfsql.exceptions import QueryExecutionException
      5

ImportError: cannot import name 'basestring' from 'pyparsing' (./miniconda3/lib/site-packages/pyparsing/__init__.py)

YarShev avatar Oct 28 '21 07:10 YarShev

Thanks.

This is fixed in https://github.com/mindsdb/dfsql/pull/39, but the mysterious Ray/Dask issues are preventing me from releasing the fix. You can checkout basestring_fix if you want to test dfsql without this problem

btseytlin avatar Oct 28 '21 10:10 btseytlin

Yes, I do not see the issue related to the import error in basestring_fix branch but it would be great to have a new release of dfsql to support it in Modin as we normally do.

YarShev avatar Oct 28 '21 11:10 YarShev

Yes, I do not see the issue related to the import error in basestring_fix branch but it would be great to have a new release of dfsql to support it in Modin as we normally do.

Absolutely! But currently dfsql doesn't pass tests on OSX with Dask engine and on Ubuntu with Ray engine. Can't make a release until this is resolved

btseytlin avatar Oct 28 '21 14:10 btseytlin

Hope this will happen soon.

YarShev avatar Oct 29 '21 06:10 YarShev

@btseytlin, what is the status on this issue?

YarShev avatar Mar 28 '22 08:03 YarShev

@btseytlin, what is the status on this issue?

CC @StpMax

btseytlin avatar Mar 28 '22 10:03 btseytlin

@StpMax can we close?

tomhuds avatar May 25 '23 12:05 tomhuds