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

Python Extension Suggestion for macOS system Python is not actionable #1689

Closed
EdBlankenship opened this issue May 15, 2018 · 5 comments · Fixed by #2570
Closed

Python Extension Suggestion for macOS system Python is not actionable #1689

EdBlankenship opened this issue May 15, 2018 · 5 comments · Fixed by #2570
Assignees
Labels
feature-request Request for new features or functionality

Comments

@EdBlankenship
Copy link

Issue Type: Bug

The Python extension is giving me a suggestion but doesn't tell me how to do this. "Selected interpreter is macOS system Python which is not recommended. Please select different interpreter."

VS Code version: Code 1.23.1 (d0182c3417d225529c6d5ad24b7572815d0de9ac, 2018-05-10T16:03:31.083Z)
OS version: Darwin x64 17.5.0

System Info
Item Value
CPUs Intel(R) Core(TM) i7-6820HQ CPU @ 2.70GHz (8 x 2700)
GPU Status 2d_canvas: enabled
flash_3d: enabled
flash_stage3d: enabled
flash_stage3d_baseline: enabled
gpu_compositing: enabled
multiple_raster_threads: enabled_on
native_gpu_memory_buffers: enabled
rasterization: enabled
video_decode: enabled
video_encode: enabled
vpx_decode: enabled
webgl: enabled
webgl2: enabled
Load (avg) 2, 2, 2
Memory (System) 16.00GB (0.03GB free)
Process Argv /Applications/Visual Studio Code.app/Contents/MacOS/Electron -psn_0_16969774
Screen Reader no
VM 0%
Extensions (15)
Extension Author (truncated) Version
vscode-docker Pet 0.0.26
scala dal 0.0.5
vscode-dash dee 1.10.0
php-debug fel 1.12.3
php-intellisense fel 2.3.1
composer ika 0.7.1
mssql ms- 1.3.1
python ms- 2018.4.0
azure-account ms- 0.4.0
team ms- 1.133.0
debugger-for-chrome msj 4.4.3
laravel-blade one 1.18.0
laravel5-snippets one 1.4.0
laravel-helpers raf 0.2.2
laravel-artisan rya 0.0.20

2018-05-15_10-32-52

@brettcannon
Copy link
Member

You can either click down on the status bar where your interpreter is listed or run Select Interpreter and then select another option. I will leave this open, though, to consider tweaking the message.

@brettcannon brettcannon added feature-request Request for new features or functionality needs decision labels May 15, 2018
@brettcannon brettcannon added this to the June 2018 milestone May 30, 2018
@brettcannon
Copy link
Member

We should provide a button to open our docs on how to handle this or something.

@brettcannon brettcannon removed this from the June 2018 milestone Jun 4, 2018
@qubitron
Copy link

qubitron commented Sep 8, 2018

Proposed design
If the only Python interpreter available is the macOS system install, the text should say:

The macOS system install of Python is not recommended, some functionality
in the extension will be limited. Install another version of Python for the 
best experience.

[ Learn more ]

Learn more should link to: https://code.visualstudio.com/docs/python/python-tutorial#_prerequisites

If there are multiple versions of Python installed

  • We should NOT select the macOS install of python by default
  • If the user selects the macOS system install, we show the following message:
You have selected the macOS system install of Python, which is not not 
recommended for use with the Python extension. Some functionality will
be limited, please select a different interpreter.

[ Select Interpreter ]

Select interpreter opens up the select python interpreter command.

@brettcannon
Copy link
Member

This is working as you outlined, @qubitron , so just need you to sign off.

@DonJayamanne
Copy link

@brettcannon This is not working.

@lock lock bot locked as resolved and limited conversation to collaborators Nov 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants