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

DiagrammeR::grViz() in chunk not working in new versions of Quarto #218

Closed
asadow opened this issue Nov 18, 2024 · 5 comments
Closed

DiagrammeR::grViz() in chunk not working in new versions of Quarto #218

asadow opened this issue Nov 18, 2024 · 5 comments

Comments

@asadow
Copy link

asadow commented Nov 18, 2024

After updating Quarto and rendering my .qmd, I got this error output instead of the diagram: TypeError: Assignment to constant variable.

I had to downgrade back to 1.3.450 to fix it.

Image

@cderv
Copy link
Collaborator

cderv commented Nov 19, 2024

After updating Quarto and rendering my .qmd

Can you share you .qmd please ? This would allow to have a closer look. Or just an example of what you are doing.

Otherwise, I can only guess what you are doing. Thank you

@asadow
Copy link
Author

asadow commented Nov 19, 2024

example.qmd

```{r}
DiagrammeR::grViz("digraph {

graph [layout = dot, rankdir = LR]
node [shape = rectangle, style = filled, fillcolor = Linen]
owner [label = 'Owner', shape = circle]

}")

@cderv
Copy link
Collaborator

cderv commented Nov 21, 2024

How do you render ? Which quarto version to you have ? Which version of the R package .

I can't reproduce

---
title: Test
keep-md: true
---

```{r}
DiagrammeR::grViz("digraph {

graph [layout = dot, rankdir = LR]
node [shape = rectangle, style = filled, fillcolor = Linen]
owner [label = 'Owner', shape = circle]

}")

```

Image

I used latest pre-release Quarto 1.6.37, with R 4.4.2 and those packages versions

> lapply(c("knitr", "rmarkdown", "DiagrammeR"), packageVersion, )
[[1]]
[1] ‘1.49’

[[2]]
[1] ‘2.29’

[[3]]
[1] ‘1.0.11’

@asadow
Copy link
Author

asadow commented Nov 21, 2024

Huh, strange, I was using the same versions..

I suppose I'll close for now. I'm glad it works on your end.

@asadow asadow closed this as completed Nov 21, 2024
@cderv
Copy link
Collaborator

cderv commented Nov 22, 2024

If it does not work for you, then we need to find why. So don't hesitate to reopen and share more information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants