Code cells in .R files

[ElianHugh]

Implements jupyter code block style decorators for .R files, piggybacking on the previous rmarkdown code lens implementation.

What problem did you solve?

Closes #191

Screenshot

(If you do not have screenshot) How can I check this pull request?

Open .R file, and make a code block like so:

# %% I am a chunk
mtcars
# %%

# %% So am I
2 * 2
# %%

Markdown code fences should not work in .R files, and the new decorator should not work in rmd files

[renkun-ken]

Is there a difference using document.languageId instead of languages.match('r', document) > 0?

[ElianHugh]

I haven't found any difference, thanks for picking that up! Not sure what the use case would be if languageId exists

[renkun-ken]

languages seems unnecessary now.

[renkun-ken]

LGTM

[apoorvalal]

Thanks for implementing this! Was wondering if it wouldn't be too much trouble to make cell delimiters single #%%s instead of requiring both top and bottom delimiters.

For example, percent scripts produced by jupytext currently skip alternate cells because the #%% at the bottom of a the i-th cell isn't interpreted as the beginning of the i+1-th cell.

I'm assuming this is because the current implementation replaces the rmarkdown ``` delimiter with #%%, so it expects both top and bottom fences.

Also, seems like the shortcut to run chunk doesn't work for R scripts fenced by #%%; it runs the entire script when i use Ctrl+Shift+Enter (run chunk shortcut on linux). Clicking on Run Chunk, however, produces the desired behaviour (i.e. it runs just the current cell).

[ElianHugh]

Thank you for the feedback! Just to be 100% clear on expected functionality, would it work like the following:

# %% 
chunk 1

# %%
chunk 2




still chunk 2

i.e., a chunk continues until a new chunk is defined via # %%?

[apoorvalal]

Yep, exactly. Since there are no text blocks in scripts, there's no need for a separate bottom delimiter.

[Yunuuuu]

Thanks for this excellent feature. Can the background between both # %% be customized?

[renkun-ken]

Thanks for this excellent feature. Can the background between both # %% be customized?

r.rmarkdown.chunkBackgroundColor.

[Yunuuuu]

Thanks! renkun! I didn't realize this is the same with rmarkdown. In this case, the problem raised in #901, which indicated Ctrl + Shift + ENTER couldn't work to run current chunk, is because of the when clause:

[renkun-ken]

Could you try changing the when clause to editorTextFocus && (editorLangId == 'r' || editorLangId == 'rmd') and see if it works?

[Yunuuuu]

yeah, this can work. I have tried it last night. Just post here to let @ElianHugh know.

[renkun-ken]

@ElianHugh Looks like we should allow all these commands to run in both r and rmd?

[Yunuuuu]

@renkun-ken, thanks for your help. I find the paranthesis is not supported in when clause, as pointed in microsoft/vscode#91473. so the when clause here should be editorLangId == r && editorTextFocus || editorLangId == rmd && editorTextFocus

[ElianHugh]

Seems like a good change to me! I'd like to push this along with some of the jupyter code cell changes, but feel free to push a change sooner if needed