quarto-dev / quarto-dev/quarto-cli

ojs_define from pytthon creates a cell (even with python has output and echo set to false)

Open
#11,495 2 comments 0 reactions 1 assignee View on GitHub

@cscheid is already working on this.

Since Nov 21, 2024.

bug dashboards ojs-engine
Dominant language
JavaScript
Stars
6k
Forks
458
Avg merge
1d 9h
Merged PRs (30d)
41

Description

Bug description
#| output: false
#| echo: false
ojs_define(value = 0)

You can see the div here in firefox

Image

The unfortunately messes up dashboard rendering, and as this is usually at the top of files (to process variables before loading ojs) it pushes everything down a bit.

Steps to reproduce
#| output: false
#| echo: false
ojs_define(value = 0)
Expected behavior

No new div should be created, as output/echo are both false

Actual behavior

No response

Your environment

No response

Quarto check output
Quarto 1.6.37
[>] Checking environment information...
      Quarto cache location: C:\Users\Joey\AppData\Local\quarto
[>] Checking versions of quarto binary dependencies...
      Pandoc version 3.4.0: OK
      Dart Sass version 1.70.0: OK
      Deno version 1.46.3: OK
      Typst version 0.11.0: OK
[>] Checking versions of quarto dependencies......OK
[>] Checking Quarto installation......OK
      Version: 1.6.37
      Path: E:\Quarto\bin
      CodePage: 1252

[>] Checking tools....................OK
      TinyTeX: v2024.03
      Chromium: (not installed)

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

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

[>] Checking Python 3 installation....OK
      Version: 3.12.7 (Conda)
      Path: C:/Development/FutureBeesStuff/.pixi/envs/default/python.exe
      Jupyter: 5.7.2
      Kernels: python3

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

[>] Checking R installation...........OK
      Version: 4.4.1
      Path: C:/Development/FutureBeesStuff/.pixi/envs/default/lib/R
      LibPaths:
        - C:/Development/FutureBeesStuff/.pixi/envs/default/lib/R/library
      knitr: (None)
      rmarkdown: (None)

      The knitr package is not available in this R installation.
      Install with install.packages("knitr")
      The rmarkdown package is not available in this R installation.
      Install with install.packages("rmarkdown")

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.