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.

Sign up for GitHub

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

Jump to bottom

Select python quickpick shows ProgramFiles\WindowsApps version of python which cannot be run #20603

Closed
karthiknadig opened this issue Jan 31, 2023 · 4 comments · Fixed by #20613
Closed

Select python quickpick shows ProgramFiles\WindowsApps version of python which cannot be run #20603

karthiknadig opened this issue Jan 31, 2023 · 4 comments · Fixed by #20613
Assignees
@karrtikr
Labels
area-environments Features relating to handling interpreter environments bug Issue identified by VS Code Team member as probable bug needs PR Ready to be worked on verification-needed Verification of issue is requested verified Verification succeeded windows
Milestone
February 2023

Comments

@karthiknadig
Copy link
Member

image
@karthiknadig karthiknadig added bug Issue identified by VS Code Team member as probable bug windows labels Jan 31, 2023
@github-actions github-actions bot added the triage-needed Needs assignment to the proper sub-team label Jan 31, 2023
@karrtikr
Copy link

karrtikr commented Jan 31, 2023 •
edited
Loading

Should've been fixed with #20457. Would need:

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python) 

XXX

@karthiknadig
Copy link
Member Author 

Experiment 'pythonSurveyNotification' is active
Experiment 'pythonPromptNewToolsExt' is active
LSP Notebooks experiment is disabled -- Jupyter disabled or not installed
LSP Notebooks interactive window support is disabled -- not in LSP Notebooks experiment
Python interpreter path: .\.venv\Scripts\python.exe
Send text to terminal: "c:/GIT/s p/vscode-python/.venv/Scripts/activate.bat"
Starting Pylance language server.
> conda info --json
> C:\Python311\python.exe -I ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\get_output_via_markers.py ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\interpreterInfo.py
> "c:\GIT\s p\vscode-python\.venv\Scripts\python.exe" -I ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\get_output_via_markers.py ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\interpreterInfo.py
> ~\AppData\Local\Microsoft\WindowsApps\python3.10.exe -I ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\get_output_via_markers.py ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\interpreterInfo.py
> ~\AppData\Local\Microsoft\WindowsApps\python3.11.exe -I ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\get_output_via_markers.py ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\interpreterInfo.py
> ~\AppData\Local\Microsoft\WindowsApps\python3.8.exe -I ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\get_output_via_markers.py ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\interpreterInfo.py
> ~\AppData\Local\Microsoft\WindowsApps\python3.9.exe -I ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\get_output_via_markers.py ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\interpreterInfo.py
> "C:\Program Files\Python37\python.exe" -I ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\get_output_via_markers.py ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\interpreterInfo.py
> "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.11_3.11.496.0_x64__qbz5n2kfra8p0\python3.11.exe" -I ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\get_output_via_markers.py ~\.vscode-insiders\extensions\ms-python.python-2023.1.10271009\pythonFiles\interpreterInfo.py

@karrtikr karrtikr added area-environments Features relating to handling interpreter environments needs PR Ready to be worked on and removed triage-needed Needs assignment to the proper sub-team labels Jan 31, 2023
@karrtikr karrtikr added this to the February 2023 milestone Jan 31, 2023
@karrtikr
Copy link

Debugged offline to find out it was coming from the windows registry.

@karrtikr karrtikr mentioned this issue Jan 31, 2023
Merged
karrtikr pushed a commit that referenced this issue Jan 31, 2023
Skip windows store interpreters found via windows registry when disco…
bf4091e 
…vering (#20613)

Closes #20603
@karthiknadig karthiknadig added verification-steps-needed Steps to verify are needed for verification verification-needed Verification of issue is requested verified Verification succeeded and removed verification-steps-needed Steps to verify are needed for verification labels Feb 21, 2023
@karthiknadig
Copy link
Member Author

image

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 24, 2023
wesm pushed a commit to posit-dev/positron that referenced this issue Mar 28, 2024
@karthiknadig @eleanorjboyd @petetronic
Merge remote-tracking branch 'upstream/main'
50a5d67 
Merging upstream vscode-python from 2023-02-03 commit fe4c5f1e4ae069160514495a717ff46f4cd11a61

--------------------
Commit message for microsoft/vscode-python@fe4c5f1:

Add a button in prompt to check logs in case the selected interpreter is invalid (microsoft/vscode-python#20642)


--------------------
Commit message for microsoft/vscode-python@1538833:

Allow to select a Python2.7 interpreter (microsoft/vscode-python#20639)


--------------------
Commit message for microsoft/vscode-python@9271136:

Move `replaceAll` and `splitLines` out of string extensions. (microsoft/vscode-python#20626)

For microsoft/vscode-python#18871
--------------------
Commit message for microsoft/vscode-python@e743037:

Ensure editable install only when [build-system] is present `pyproject.toml` (microsoft/vscode-python#20625)

Fixes microsoft/vscode-python#20620
--------------------
Commit message for microsoft/vscode-python@b43bc25:

Do not recommend Python2 as an interpreter (microsoft/vscode-python#20628)


--------------------
Commit message for microsoft/vscode-python@ddc765e:

Fix wording in conda inherit env prompt (microsoft/vscode-python#20627)

Closes microsoft/vscode-python#19001
--------------------
Commit message for microsoft/vscode-python@401418a:

Ensure IDs of Conda environments without python does not change after python is installed (microsoft/vscode-python#20609)

Fixes microsoft/vscode-python#20176

As we're changing IDs we've to ensure the same environment in cache with
older ID is migrated to use the new one, this is already ensured here:
https://github.com/microsoft/vscode-python/blob/32f55109c976e66bf39e8da6aae0c9b6f5115df2/src/client/pythonEnvironments/base/locators/composite/envsCollectionCache.ts#L109
--------------------
Commit message for microsoft/vscode-python@92d2d85:

Fix test failures due to linting (microsoft/vscode-python#20617)

Fixes microsoft/vscode-python#20612
--------------------
Commit message for microsoft/vscode-python@d245a90:

Update to latest `jedi-language-server` (microsoft/vscode-python#20611)

Fixes microsoft/vscode-python#20606
--------------------
Commit message for microsoft/vscode-python@3fe7057:

new inactive discovery logic (microsoft/vscode-python#20566)

New pytest code in a currently inactive state. 
--------------------
Commit message for microsoft/vscode-python@bf4091e:

Skip windows store interpreters found via windows registry when discovering (microsoft/vscode-python#20613)

Closes microsoft/vscode-python#20603
--------------------
Commit message for microsoft/vscode-python@ec2af99:

Use `markdownDescription` for settings with markdown in description (microsoft/vscode-python#20600)

Fixes microsoft/vscode-python#20582
--------------------
Commit message for microsoft/vscode-python@7ca872e:

Add option to skip showing recommended interpreter in quick pick API (microsoft/vscode-python#20604)

Closes microsoft/vscode-python#20260
--------------------
Commit message for microsoft/vscode-python@184617e:

Check to ensure major version matches current year. (microsoft/vscode-python#20601)

Fixes microsoft/vscode-python#20466
--------------------
Commit message for microsoft/vscode-python@328e511:

Ensure dependency quick pick is not shown when user exits env creation. (microsoft/vscode-python#20605)

Fixes microsoft/vscode-python#20602
--------------------
Commit message for microsoft/vscode-python@7fb72b7:

Update main to next pre-release (microsoft/vscode-python#20598)


--------------------
Commit message for microsoft/vscode-python@2dd9287:

Set release candidate version. (microsoft/vscode-python#20597)


--------------------
Commit message for microsoft/vscode-python@b83aacb:

Update `debugpy` to v1.6.6 (microsoft/vscode-python#20596)


--------------------
Commit message for microsoft/vscode-python@93b0053:

Skip requirement files from `site-packages` or `__pypackages__` (microsoft/vscode-python#20573)

Fixes microsoft/vscode-python#20560

Lead-authored-by: Karthik Nadig <kanadig@microsoft.com>
Co-authored-by: Eleanor Boyd <eleanorboyd@microsoft.com>
Co-authored-by: Pete Farland <pete.farland@posit.co>
Co-authored-by: Kartik Raj <karraj@microsoft.com>
wesm pushed a commit to posit-dev/positron that referenced this issue Mar 28, 2024
@karthiknadig @eleanorjboyd
@petetronic @wesm
Merge remote-tracking branch 'upstream/main'
06c5dad 
Merging upstream vscode-python from 2023-02-03 commit fe4c5f1e4ae069160514495a717ff46f4cd11a61

--------------------
Commit message for microsoft/vscode-python@fe4c5f1:

Add a button in prompt to check logs in case the selected interpreter is invalid (microsoft/vscode-python#20642)


--------------------
Commit message for microsoft/vscode-python@1538833:

Allow to select a Python2.7 interpreter (microsoft/vscode-python#20639)


--------------------
Commit message for microsoft/vscode-python@9271136:

Move `replaceAll` and `splitLines` out of string extensions. (microsoft/vscode-python#20626)

For microsoft/vscode-python#18871
--------------------
Commit message for microsoft/vscode-python@e743037:

Ensure editable install only when [build-system] is present `pyproject.toml` (microsoft/vscode-python#20625)

Fixes microsoft/vscode-python#20620
--------------------
Commit message for microsoft/vscode-python@b43bc25:

Do not recommend Python2 as an interpreter (microsoft/vscode-python#20628)


--------------------
Commit message for microsoft/vscode-python@ddc765e:

Fix wording in conda inherit env prompt (microsoft/vscode-python#20627)

Closes microsoft/vscode-python#19001
--------------------
Commit message for microsoft/vscode-python@401418a:

Ensure IDs of Conda environments without python does not change after python is installed (microsoft/vscode-python#20609)

Fixes microsoft/vscode-python#20176

As we're changing IDs we've to ensure the same environment in cache with
older ID is migrated to use the new one, this is already ensured here:
https://github.com/microsoft/vscode-python/blob/32f55109c976e66bf39e8da6aae0c9b6f5115df2/src/client/pythonEnvironments/base/locators/composite/envsCollectionCache.ts#L109
--------------------
Commit message for microsoft/vscode-python@92d2d85:

Fix test failures due to linting (microsoft/vscode-python#20617)

Fixes microsoft/vscode-python#20612
--------------------
Commit message for microsoft/vscode-python@d245a90:

Update to latest `jedi-language-server` (microsoft/vscode-python#20611)

Fixes microsoft/vscode-python#20606
--------------------
Commit message for microsoft/vscode-python@3fe7057:

new inactive discovery logic (microsoft/vscode-python#20566)

New pytest code in a currently inactive state. 
--------------------
Commit message for microsoft/vscode-python@bf4091e:

Skip windows store interpreters found via windows registry when discovering (microsoft/vscode-python#20613)

Closes microsoft/vscode-python#20603
--------------------
Commit message for microsoft/vscode-python@ec2af99:

Use `markdownDescription` for settings with markdown in description (microsoft/vscode-python#20600)

Fixes microsoft/vscode-python#20582
--------------------
Commit message for microsoft/vscode-python@7ca872e:

Add option to skip showing recommended interpreter in quick pick API (microsoft/vscode-python#20604)

Closes microsoft/vscode-python#20260
--------------------
Commit message for microsoft/vscode-python@184617e:

Check to ensure major version matches current year. (microsoft/vscode-python#20601)

Fixes microsoft/vscode-python#20466
--------------------
Commit message for microsoft/vscode-python@328e511:

Ensure dependency quick pick is not shown when user exits env creation. (microsoft/vscode-python#20605)

Fixes microsoft/vscode-python#20602
--------------------
Commit message for microsoft/vscode-python@7fb72b7:

Update main to next pre-release (microsoft/vscode-python#20598)


--------------------
Commit message for microsoft/vscode-python@2dd9287:

Set release candidate version. (microsoft/vscode-python#20597)


--------------------
Commit message for microsoft/vscode-python@b83aacb:

Update `debugpy` to v1.6.6 (microsoft/vscode-python#20596)


--------------------
Commit message for microsoft/vscode-python@93b0053:

Skip requirement files from `site-packages` or `__pypackages__` (microsoft/vscode-python#20573)

Fixes microsoft/vscode-python#20560

Lead-authored-by: Karthik Nadig <kanadig@microsoft.com>
Co-authored-by: Eleanor Boyd <eleanorboyd@microsoft.com>
Co-authored-by: Pete Farland <pete.farland@posit.co>
Co-authored-by: Kartik Raj <karraj@microsoft.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@karrtikr karrtikr

Labels
area-environments Features relating to handling interpreter environments bug Issue identified by VS Code Team member as probable bug needs PR Ready to be worked on verification-needed Verification of issue is requested verified Verification succeeded windows
Projects
None yet
Milestone
February 2023
Development

Successfully merging a pull request may close this issue.

Skip windows store interpreters found via windows registry when discovering karrtikr/vscode-python
2 participants
@karthiknadig @karrtikr