Code cells of different languages are wrongly merged when inside a layout

[mcanouil]

Code cells of different languages are wrongly merged when inside a layout (independently of knitr/Jupyter being the engine)

Quarto document	v1.3.450	v1.4
--- title: "Block layout" format: html engine: knitr --- # Code chunk block layout :::: {layout="[48,-4,48]"} ::: {} ### R ```{r} sqrt(2) ``` ::: ::: {} ### Python ```{python} import math math.sqrt(2) ``` ::: ::::

Discussed in #8178

^{Originally posted by luifrancgom January 9, 2024}

Description

I want to display code chunks side by side where I try the solution specify in #3423 and the instructions pointed out in the documentation here but I did not obtain the expected behavior in relation to the code. The solution only works for the result of the code.

Steps to reproduce

---
title: "Block layout"
format: 
  html: default
  pdf: default
editor: visual
---

# Code chunk block layout

:::: {layout="[48,-4,48]"}

::: {}

### R

```{r}
sqrt(2)
```

:::

::: {}

### Python

```{python}
import math
math.sqrt(2)
```

:::

::::

Expected behavior

Actual behavior

Question

How can I obtain the expected behavior?

Quarto and dependencies I am running

Quarto 1.4.537
[>] Checking versions of quarto binary dependencies...
      Pandoc version 3.1.11: OK
      Dart Sass version 1.69.5: OK
      Deno version 1.37.2: OK
[>] Checking versions of quarto dependencies......OK
[>] Checking Quarto installation......OK
      Version: 1.4.537
      Path: C:\Users\Usuario\AppData\Local\Programs\Quarto\bin
      CodePage: 1252

[>] Checking tools....................OK
      TinyTeX: v2024.01
      Chromium: 869685

[>] Checking LaTeX....................OK
      Using: TinyTex
      Path: C:\Users\Usuario\AppData\Roaming\TinyTeX\bin\windows\
      Version: 2023

[>] Checking basic markdown render....OK

[>] Checking Python 3 installation....OK
      Version: 3.11.6 (Conda)
      Path: C:/Users/Usuario/anaconda3/python.exe
      Jupyter: 5.7.0
      Kernels: ir, julia-1.9, python3

[>] Checking Jupyter engine render....OK

[>] Checking R installation...........OK
      Version: 4.3.2
      Path: C:/PROGRA~1/R/R-43~1.2
      LibPaths:
        - C:/Users/Usuario/AppData/Local/R/win-library/4.3
        - C:/Program Files/R/R-4.3.2/library
      knitr: 1.45
      rmarkdown: 2.25

[>] Checking Knitr engine render......OK
```</div>

[cderv]

FWIW this is the intermediate .md

# Code chunk block layout

:::: {layout="[48,-4,48]"}

::: {}

### R


::: {.cell}

```{.r .cell-code}
sqrt(2)
```

::: {.cell-output .cell-output-stdout}

```
[1] 1.414214
```


:::
:::


:::

::: {}

### Python


::: {.cell}

```{.python .cell-code}
import math
math.sqrt(2)
```

::: {.cell-output .cell-output-stdout}

```
1.4142135623730951
```


:::
:::


:::

::::

Also this is not related to knitr as this happens with Jupyter engine too

---
title: "Block layout"
format: html
keep-md: true
---

# Code chunk block layout

:::: {layout="[48,-4,48]"}

::: {}

### Python

```{python}
import math
math.sqrt(4)
```

:::

::: {}

### Python

```{python}
import math
math.sqrt(2)
```

:::

::::

So there seems to be indeed an issue with cell merging. Example can be simplified with

execute: 
  eval: false

[cderv]

This is the change that created the regression c4ab028 from

• layouts: hoist deep code cells into layout preamble. #7598 to solve PDF side-by-side figure with caption error with echo: false #4370

I did a git bisect to confirm

[mcanouil]

@cderv thanks for the bisect (4am is not a good time to do it^^)

[cderv]

No need to justify. And no need to do triaging at 4AM too. 😉

[mcanouil]

@cderv Well, it was no time wasted 😉

[cscheid]

Let me start by the first comment. The "expected" vs "actual" behavior goes the opposite way. We've always wanted to merge the code from layout cells but weren't doing it in all situations. This is now working as expected, as isn't a bug.

[cscheid]

We might want to add an option to not to code hoisting, but this isn't a bug.

[mcanouil]

Well, the two code cells are of different languages. Only code cells sharing the same language should be merged at least.
Currently, no matter the code cells languages, those are merged.

[cscheid]

I see. We need to edit the issue title and description.