vscode-R icon indicating copy to clipboard operation
vscode-R copied to clipboard

R executable selection

Open ElianHugh opened this issue 2 years ago • 5 comments

WIP R executable selection

Major contribution: extended R executable selection. Related issues #696 #1130 #1148 #747

This PR may need to be split into two parts: (1) R executable selection and (2) improved multi-root support

Screenshots

image

image

image

TODO

R Executable

  • [x] Language status item
  • [x] Executable quick pick
  • [x] Unix support for standard R installations
  • [ ] Unix support for all virtual environments
  • [x] Windows support for standard R installations
  • [ ] Windows support for all virtual environments
  • [ ] Look into better supporting renv

Multi-root

Multi-root executable support potentially requires modification of many vscode-R features

  • [x] R terminal
  • [ ] Language service
  • [ ] Help provider
  • [x] Tasks

Other features should probably be deferred to another PR

ElianHugh avatar Jul 04 '22 12:07 ElianHugh

I'll have access to windows later today, so I should be able to work on the windows side to get to feature parity

ElianHugh avatar Jul 13 '22 02:07 ElianHugh

An issue I've found that on windows side, if an R executable isn't openable it causes an annoying popup when loading the extension. It doesn't look like fs.access or fs.stat checks for this on windows

ElianHugh avatar Jul 14 '22 04:07 ElianHugh

This is currently how the picker is looking:

image

Recommendation is due to the renv lock file denoting a matching R version

ElianHugh avatar Jul 15 '22 03:07 ElianHugh

Coming back to this PR, I think it's probably okay if support for "all" virtual environments is scaled back to just conda and renv?

ElianHugh avatar Sep 07 '22 05:09 ElianHugh

if support for "all" virtual environments is scaled back to just conda and renv?

I think it's almost the same as conda, but also mamba?

eitsupi avatar Sep 07 '22 05:09 eitsupi

I'm reasonably happy with the executable UI & the executable locator services. main issue for this PR right now are conda activation (seems a little fragile in terms of how I'm handling it) and also feature parity on windows devices

ElianHugh avatar Feb 28 '23 02:02 ElianHugh

I'm waiting eagerly for the R executable selection with conda/mamba support. Any chance to have this PR finalized & merged in the near future?

floriandeboissieu avatar Nov 27 '23 23:11 floriandeboissieu

Hi @floriandeboissieu, thank you for your interest in the PR. Unfortunately this year has been pretty chock-a-block and I haven't had much time to do open source projects unless they are directly related to work. Having said that, I will have plenty of free time over the Christmas break, so I'm hoping to jump back into things then.

ElianHugh avatar Nov 29 '23 23:11 ElianHugh

Thank you all for the feedback. I am closing this PR in favour of work on a new branch (#1473).

ElianHugh avatar Jan 05 '24 03:01 ElianHugh