-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
RecursionError: maximum recursion depth exceeded while calling a Python object #3746
Comments
We are also experiencing this with the same versions of pylint, astroid and python as listed above. For us it seems to have started happening when pandas updated from 1.0.5 to 1.1.0. |
I can't reproduce this, although pylint does get stuck at some point in the linting process so it may well be recursing deep at that point. My recursion limit is at 1000: >>> import sys
>>> print(sys.getrecursionlimit())
1000 |
I can't reproduce the error with the initial script, however I have a slightly different (minimal) script causing the same error import pandas
def process(dfa: pandas.DataFrame, dfb: pandas.DataFrame) -> pandas.DataFrame:
return pandas.merge(dfa, dfb) $ pylint --disable=all --enable=F,E mwe.py
[...]
RecursionError: maximum recursion depth exceeded
The relevant versions $ pylint --version
pylint 2.6.0
astroid 2.4.2
Python 3.8.2 (default, Jul 16 2020, 14:00:26)
[GCC 9.3.0]
$ pip freeze | grep pandas
pandas==1.1.2 When downgrading $ pip install pandas~=1.0.0
[...]
$ pip freeze | grep pandas
pandas==1.0.5
$ pylint --disable=all --enable=F,E mwe.py
--------------------------------------------------------------------
Your code has been rated at 10.00/10 (previous run: 0.00/10, +10.00)
|
@mberr thanks for this update. I can reproduce the bug with your snippet.
|
@mberr this issue is similar to #2843 and #2811. Thanks to @brycepg and pylint-dev/astroid#804 it is solved in the master branch of |
I and can still reproduce this in python 3.7 and astroid at current master. |
@adam-azarchs thanks for your feedback. Can you give us a snippet of the problem please? |
Code similar to that in #3746 (comment) using |
I'm seeing this problem with the python:3.8 Docker file: https://gist.github.com/bnsh/b959f91a33ac014e41c1692e9269e1e7 The gist shows the output I get.
|
@adam-azarchs in fact the pandas version you use is quite old. It would be nice if you could test with a more recent version. |
Steps to reproduce
Current behavior
Crashes with
RecursionError: maximum recursion depth exceeded while calling a Python object
Expected behavior
Not to crash
pylint --version output
pylint 2.5.3
astroid 2.4.2
Python 3.8.2
GCC 9.3.0
extra notes
This crash does not occur with pylint==2.3.1 & astroid==2.2.5 similar to #3318
The text was updated successfully, but these errors were encountered: