-
Notifications
You must be signed in to change notification settings - Fork 770
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
"RangeError: Maximum call stack size exceeded" #5090
Comments
I'm not able to repro the problem as reported. If someone on the pylance team is able to repro, please include repro steps and (if appropriate) move to the pyright project. |
I can't reproduce the problem either. Typing in that code manually seems to work fine with Pylance 2023.11.10. On reload it still opens. @whoahbot - The callstack seems to indicate a completion was being created. Are there any more repro steps that are necessary? Like maybe you get the crash when typing the import? |
Hi, thanks for having a look at this. I just tried reproducing the error again, and yes, you are correct. The error was triggered in my new session when I opened the code snippet I provided and tried adding a second import line after the first: from bytewax.dataflow import Dataflow
from bytewax.i # <-- crash log appears here The module |
…ize and parse a binary (native) library file. This led to a crash in some cases. This addresses microsoft/pylance-release#5090.
Thanks for the bug report. This is due to a bug in the completion provider component. It was improperly allowing binary (native) library files to be treated as Python source files. An attempt was then made to read the contents of the binary and tokenize/parse it. In some cases, this could result in a crash. I've fixed the logic so it no longer attempts to treat these binaries as source files. @rchiodo, I've posted a PR for your review. Apologies if you were already working on a fix. I just noticed that you had assigned this bug to yourself. |
…ize and parse a binary (native) library file. This led to a crash in some cases. This addresses microsoft/pylance-release#5090. (#6356)
* Fixed bug in completion provider that resulted in an attempt to tokenize and parse a binary (native) library file. This led to a crash in some cases. This addresses microsoft/pylance-release#5090. * Fixed a bug that results in a false positive error when a `__new__` method returns `Self` for a generic class and the type variable is invariant. This addresses #6335.
No worries, thanks for fixing it :) |
This issue has been fixed in prerelease version 2023.11.12, which we've just released. You can find the changelog here: CHANGELOG.md |
Environment data
Code Snippet
Repro Steps
pip install bytewax
Expected behavior
Pylance should show inline documentation.
Actual behavior
Pylance crashes.
Logs
/cc bytewax/bytewax/issues/187
The text was updated successfully, but these errors were encountered: