-
Notifications
You must be signed in to change notification settings - Fork 133
Intellisense doesnt work for all Pytorch methods #1098
Comments
The PyTorch methods mentioned here are dynamically generated, so static analysis won't pick them up. PyTorch is now shipped with stub files which VSCode correctly uses to provide intellisense, so I think this can be closed. |
Great to hear :-) |
@malmaud : I noticed other problems as well. such as attributes/methods not showing up concerning tensors (such as
sometimes works and sometimes doesnt provide intellisense!
now both |
#1301 is primary |
0.4.26+ |
@Coderx7 commented on Wed May 15 2019
Environment data
Expected behaviour
Intellisense should work on Pytorch methods as well but it does not!
Actual behaviour
It wont work on methods such as
mm
,matmul
,mul
,from_numpy
,sum
,randn
,tensor
,exp
,empty
arange
,zeros
,cat
,etc as described here in details.Steps to reproduce:
sum
,empty
,randn
,tensor
,etcLogs
Output for
Python
in theOutput
panel (View
→Output
, change the drop-down the upper-right of theOutput
panel toPython
)@DonJayamanne commented on Thu May 16 2019
Please could you try switching to the
Language Server
.Please check here for instructions microsoft/vscode-python#3977
Let me know whether this resolves the problem for you.
@Coderx7 commented on Thu May 16 2019
@DonJayamanne : I had a look at the link you provided. the problem is I cant find a way to install Language Server, as the instructions given there doesn't work for me.
I dont have a
Language Server
folder, to remove in first place!This is how my extension directory looks like :
and this is the contents of
ms-python.python
folder :I just updated the VSCode to the latest version couple of hours ago and this has not changed (i.e. no language server is installed)
I'd appreciate any feedback on how to get that installed so I can give you the result you need.
@DonJayamanne commented on Thu May 16 2019
All you need to do is:
@Coderx7 commented on Fri May 17 2019
Thanks, I did so and language server seem to have its own share of issues, while the mentioned methods now show up in intellisense, the presentation is different and more importantly, intellisense doesnt work flawlessly here either, here is a demonstration show casing the issues:
(side note: jediEnabled presentation/look is much more desirable than what is being provided by language server! apart from its bugs! I myself prefer jediEnabled over language sever)
@DonJayamanne commented on Fri May 17 2019
Please note, you have
jedi
enabled. Please change the value tofalse
.Once done, please provide an animated gif as done previously. Thanks a lot.
@Coderx7 commented on Fri May 17 2019
@DonJayamanne In the gif I posted above, I tested both scenarios, both with jediEnabled set to true and false!
Just watch the whole gif to the end and you'll see the difference between the two.
I start with JediEnabled and show how it works, and then disable it and show how language server works
@DonJayamanne commented on Fri May 17 2019
When using Jedi, the auto completion list is provided by an external package and we do not have much control over that. Please feel free to report an issue on their repo https://github.com/davidhalter/jedi.
On the other hand the language server is something that's built by us.
Hence I'll be moving this issue to the language server repo so they can address the issues/concerns you have.
The text was updated successfully, but these errors were encountered: