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

Method completions inserting double values when selected #5956

Closed
SiddhanthUnnithan opened this issue May 21, 2021 · 9 comments
Closed

Method completions inserting double values when selected #5956

SiddhanthUnnithan opened this issue May 21, 2021 · 9 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone

Comments

@SiddhanthUnnithan
Copy link

I'm currently working in a native notebook editor on my local VS Code.

I have my Python language server set to "None" (not using Pylance or Jedi), and am seeing the following for completions:

image

When choosing the completion I see the following:

image

Instead of just inserting "get_image_details" the Notebook editor inserts "env.get_image_details". I'd expect only the method from the completion to be inserted when selected.

@SiddhanthUnnithan SiddhanthUnnithan added the bug Issue identified by VS Code Team member as probable bug label May 21, 2021
@claudiaregio
Copy link
Contributor

Have to discuss if this is worth fixing since this is why we recommend language servers

@rchiodo
Copy link
Contributor

rchiodo commented May 27, 2021

Somehow the text edits from Jupyter aren't working in latest VS code.

I'm guessing this doesn't repro in the interactive window.

@rchiodo rchiodo added this to the July 2021 Release milestone May 27, 2021
@greazer
Copy link
Member

greazer commented Jun 6, 2021

It does not repro in the IW. However, it seems likely that people who set the LS to none is a small number. Therefore, moving out of upcoming milestone.

@jiewangjs
Copy link

Any feedback on the problem ?

@rchiodo
Copy link
Contributor

rchiodo commented Jul 26, 2021

It does not repro in the IW. However, it seems likely that people who set the LS to none is a small number. Therefore, moving out of upcoming milestone.

This repros with pylance. It doesnt' require ls set to none.

@jiewangjs
Copy link

@rchiodo Yep, I figured it out. Disabled pylance, and problem solved.

@roblourens
Copy link
Member

Was anything changed here? What needs verification?

@rzhao271 rzhao271 removed the verification-needed Verification of issue is requested label Jul 29, 2021
@SiddhanthUnnithan
Copy link
Author

@DonJayamanne just want to confirm that with the changes you merged @jiewangjs will no longer need to disable Pylance to avoid seeing double values?

@claudiaregio
Copy link
Contributor

@SiddhanthUnnithan I believe the completions were actually coming from Jupyter, we merge both Pylance and Jupyter returned ones. This wasn't a Pylance issue so @jiewangjs shouldn't need to disable Pylance for Don's fixes to be seen

@rzhao271 rzhao271 added the verified Verification succeeded label Jul 30, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

8 participants