rstudio / rstudio

RStudio is an integrated development environment (IDE) for R
https://posit.co/products/open-source/rstudio/
Other
4.68k stars 1.09k forks source link

No "Run" button for dot/graphviz engine chunks #14063

Closed jthomasmock closed 7 months ago

jthomasmock commented 10 months ago

System details

RStudio Edition : Desktop
RStudio Version : Version 2023.12.0+369 (2023.12.0+369)
OS Version      : Mac
R Version       : 4.2.0

Steps to reproduce the problem

Example code and requires local install of brew install graphviz on Mac.

```{dot}
graph G {
  layout=neato
  run -- intr;
  intr -- runbl;
  runbl -- run;
  run -- kernel;
  kernel -- zombie;
  kernel -- sleep;
  kernel -- runmem;
  sleep -- swap;
  swap -- runswap;
  runswap -- new;
  runswap -- runmem;
  new -- runmem;
  sleep -- runmem;
}


### Describe the problem in detail

While GraphViz can be run with Run All or other shortcuts, there is no "Run" green button in the chunk even in inline execution mode: 
![image](https://github.com/rstudio/rstudio/assets/29187501/82f8b73c-36b8-4957-a64a-cde555c7794d)

I'm not sure if this was intended but we do expect that https://github.com/rstudio/rstudio/pull/13745 allows for inline execution.

### Describe the behavior you expected

Given you _can_ run GraphViz inline, I'd expect a run button for that chunk.

<!-- 
Please keep the below portion in your issue, and check `[x]` the applicable boxes.
-->

- [x] I have read the guide for [submitting good bug reports](https://github.com/rstudio/rstudio/wiki/Writing-Good-Bug-Reports).
- [x] I have installed the latest version of RStudio, and confirmed that the issue still persists.
- [ ] If I am reporting an RStudio crash, I have included a [diagnostics report](https://support.posit.co/hc/en-us/articles/200321257-Running-a-Diagnostics-Report).
- [x] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
ronblum commented 10 months ago

Also the case for R Markdown. Reproducible in 2022.07.2-576 as well.

ronblum commented 7 months ago

Verified in

Check