Fix "go to definition" and resolving imports

[busykoala]

I have "python.analysis.downloadChannel": "daily" activated.

The update two days ago broke the 'go to definition' feature for my setup.
I'm using vscode and it's regarding a large plone project using the omelette as described below.

To avoid language server load infinitely (which would be another issue, maybe related to this), I excluded the parts/omelette with search.exclude.
Below that in the settings, I added the parts/omelette with: "python.autoComplete.extraPaths": ["${workspaceFolder}/parts/omelette", ], which seemed to work for the last few days now.
By 'it seemed to work' I mean it could resolve the omelette imports and jump to definitions.
Now it sais 'no definition found for...' and the imports can't be resolved anymore.

For what I can see in the output, before the update, it used to analyse from autoComplete.extraPath, which it does not anymore since it updated the language server today.

(parts/omelette contains symlinks to cross-project sources in a local directory.)

[jakebailey]

@busykoala A new build with #467 is in progress and should show up in daily in a bit.

[busykoala]

Thank you very much!

The version I tried now, could resolve imports again with given settings.

Not yet solved in my case is 'go to definition' which once was working properly (about three days ago).
The output I get when trying to jump to a definition is following:

[Error - 12:09:05 AM] Request textDocument/definition failed.
  Message: PriorityProducerConsumer`1 instance is disposed
Object name: 'PriorityProducerConsumer`1'.
  Code: -32000 
   at Microsoft.PythonTools.Analysis.Infrastructure.DisposeToken.ThrowIfDisposed()
   at Microsoft.PythonTools.Analysis.Infrastructure.PriorityProducerConsumer`1.Produce(T value, Int32 priority)
   at Microsoft.PythonTools.Intellisense.AnalysisQueue.ExecuteInQueueAsync(Func`2 function, AnalysisPriority priority)
   at Microsoft.Python.LanguageServer.Implementation.Server.WaitForCompleteAnalysisWorker(CancellationToken cancellationToken)
   at Microsoft.Python.LanguageServer.Implementation.Server.FindReferences(ReferencesParams params, CancellationToken cancellationToken)
   at Microsoft.Python.LanguageServer.Implementation.Server.GotoDefinition(TextDocumentPositionParams params, CancellationToken cancellationToken)
   at Microsoft.Python.LanguageServer.Implementation.LanguageServer.GotoDefinition(JToken token, CancellationToken cancellationToken)

Rather than this file structure some/other/package_x only is a symlink. This is pretty handy to have shared packages and is common use in plone projects.

root
│   main.py
│
├───project
│       __init__.py
│
└───some
    └───other
        └───package_x -> Users/username/.buildout/eggs/package_x

As I mentioned to avoid infinit loading on 'go to definition' it yet (and still) is necessary to search exclude **/some/other.
The folder is added again with "python.autoComplete.extraPaths":["${workspaceFolder}/some/other", ],

[jakebailey]

Is there an error message before that disposed message? That should only occur after an exception occurs in a different part of the code (#448), but the initial error should be in the same output panel.

[busykoala]

Well I think it's not really an error message except of 'no definition found for...' in the vscode gui.

Getting hover for package_name
Waiting for parsing to complete.
Parsing complete. Waiting for analysis entries to enqueue.
Enqueue complete. Waiting for analysis to complete.
[Error - 12:39:21 AM] Request textDocument/definition failed.

But it doesn't seem to scan the additional files initially. So the output makes sense.

[AlexanderSher]

@busykoala , I've tried to recreate the file structure with symlink, but for me go to definition works as expected. If you have minimal repro, can you pack it into zip or tarball and attach to the bug?

[busykoala]

I've just tried different repositories with the same structure.

In smaller project it worked having these two lines in my settings:

"python.autoComplete.extraPaths": ["${workspaceFolder}/parts/omelette", ],
"python.autoComplete.typeshedPaths": ["${workspaceFolder}/parts/omelette", ],

With bigger projects it's not working. Therefore I guess it is regarding the size of projects. I'm sorry I didn't figure that out before.

Thank you for the help and let me try python language server when it advanced a little more.