Skip to content

Ensure language server is not blocked on discovery in the first session #18671

Closed

Description

@karrtikr

If discovery takes a long time (#18200), it seems language server is not started until it has finished, even though it is not "blocked" on discovery NodeJS-wise. This prevents users from using the most basic features like the indexer, so they can't click through functions, variable definitions, etc.

In #18200 the user is blocked for about 15 mins, files are also on a shared network drive.

As language server activations takes less time, I think we can afford to postpone triggering discovery until after it has started. FYI Kim-Adeline Miguel (@kimadeline) as you're working on the new design to activate/deactivate language servers 🙂

cc/ Karthik Nadig (@karthiknadig)

Activity

  1. added
    feature-requestRequest for new features or functionality
    area-intellisenseLSP-related functionality: auto-complete, docstrings, navigation, refactoring, etc.
    area-environmentsFeatures relating to handling interpreter environments
    needs spikeLabel for issues that need investigation before they can be worked on.
    bugIssue identified by VS Code Team member as probable bug
    needs proposalNeed to make some design decisions
    and removed
    feature-requestRequest for new features or functionality
    needs spikeLabel for issues that need investigation before they can be worked on.
    on Mar 10, 2022
  2. wubdub commented on Apr 15, 2022

    @wubdub

    Friendly ping on when this might be slated for development? My team of 200 hits this every morning and at lunch when we come back to our IDEs

  3. karrtikr commented on Apr 16, 2022

    @karrtikr
    Author

    Unfortunately we haven't been able to prioritize it yet partly as it didn't get enough upvotes. Does the VSIXs provided in #18200 serve as a workaround?

  4. removed their assignment
    on Apr 16, 2022
  5. wubdub commented on Apr 19, 2022

    @wubdub

    Thanks Kartik Raj (@karrtikr), I can use the VSIX but I think it gets wiped every time I update the extension. If this post receives enough upvotes would it be prioritized?

  6. karrtikr commented on Apr 19, 2022

    @karrtikr
    Author

    It'll have much more chances of being prioritized yes.

    I can use the VSIX but I think it gets wiped every time I update the extension

    You can follow https://stackoverflow.com/a/63831525/8152786 to stop auto-updates for the Python extension.

  7. wubdub commented on May 10, 2022

    @wubdub

    It'll have much more chances of being prioritized yes.
    Kartik Raj (@karrtikr) any chance this has enough votes to be prioritized now?

  8. karrtikr commented on May 13, 2022

    @karrtikr
    Author

    #18647 received enough upvotes as well and could work for you.

  9. karrtikr commented on May 13, 2022

    @karrtikr
    Author

    wubdub #19102 now ensures auto discovery only happens in the first session and turns it off for subsequent sessions (given Jupyter is not installed). Try the latest pre-release, activate the extension, auto-discovery should no longer happen:

    image

    Let me know if that unblocks you.

  10. changed the title [-]Ensure language server is not blocked on discovery[/-] [+]Ensure language server is not blocked on discovery in the first session[/+] on Jun 3, 2022
  11. karrtikr commented on Jun 3, 2022

    @karrtikr
    Author

    This is now only a problem the first time a scope is opened, so could be an issue for the 1d users.

  12. wubdub commented on Jun 21, 2022

    @wubdub

    wubdub #19102 now ensures auto discovery only happens in the first session and turns it off for subsequent sessions (given Jupyter is not installed). Try the latest pre-release, activate the extension, auto-discovery should no longer happen:

    image

    Let me know if that unblocks you.

    Sorry for the delay in getting back to you. Running v2022.9.11681004, I still have the same problem as I originally reported, the "Discovering Python Interpreters" with a spinning icon runs and during this time no language features are available (such as click to go to definition or syntax highlighting). Like before, the output terminal has logs like > /XXX/shared/conda-envs/XXX-9f9d08bc8-990/bin/python ~/.vscode-server/extensions/ms-python.python-2022.9.11681004/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.9.11681004/pythonFiles/interpreterInfo.py and spins for 10+ mins. This is looking at one file that previously was indexed the last time VSCode was open

  13. karrtikr commented on Jun 21, 2022

    @karrtikr
    Author

    Do you have Jupyter extension installed? Note that still needs to be disabled.

  14. wubdub commented on Jun 22, 2022

    @wubdub

    Do you have Jupyter extension installed? Note that still needs to be disabled.

    Ah! Amazing I disabled Jupyter extension and now it is instant, thanks!

  15. locked as resolved and limited conversation to collaborators on Aug 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

area-environmentsFeatures relating to handling interpreter environmentsarea-intellisenseLSP-related functionality: auto-complete, docstrings, navigation, refactoring, etc.bugIssue identified by VS Code Team member as probable buginfo-neededIssue requires more information from posterneeds proposalNeed to make some design decisionsverifiedVerification succeeded

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions