Christophe Dervieux

icon indicating a website link https://cderv.rbind.io icon indicating an email [email protected]

icon company https://posit.co icon location Paris icon location software engineer at @posit-pbc working on R Markdown ecosystem and @quarto-dev

Results 1378 comments of Christophe Dervieux

Use knitr::purl() to export Python files (.py)

Could we limit the feature to document that does not use child document ? By emitting an error if child document is detected and explaining the function does not support...

New YAML-style chunk options don't process R variables or glue strings

Indeed. Thanks !

[FR] Add a new type of dependency that bundles lua filter and other type of dependencies

> I'm currently working on this. Cool ! Thank you ! > To make things simple yet powerful, I would add pandoc_dependency rather than lua_dependency. Really interesting ! Yes I...

Caching sql chunks throwing error when output.var not used

Thanks. I still don't know why we have an issue but this will be providing a base for comparison. For reminder, the issue happens in `lazyload` feature when restoring from...

Caching sql chunks throwing error when output.var not used

Oh I know now ! https://github.com/yihui/knitr/blob/039b6cfa45dea4ece31f2bcb65ed3b34b1330daa/R/block.R#L160-L163 We are specifically storing `output$output.var` when using `sql` engine as `objects` for our `block_cache()` function, which will be `NULL` when not defined, whereas for...

Caching sql chunks throwing error when output.var not used

> In other words, would it be great if wrappers for existing engines like glue_sql for sql handled cache in exactly the same way as original engines? Not sure to...

Provide a syntax definition file with updated support for R new syntax

Yes it seems ok to do that.

Package build problems with Rmd vignette using child Rmd files

> not in a pkgdown GitHub Actions workflow because there it looks for the child document at /tmp/Rtmpzf2bRh/../man/vignette_childs/ I wonder if in github action it should built with `install =...

opts_current$set should error

I believe the underlying issue is also important to fix (adapt) * https://github.com/yihui/knitr/issues/1988 as I understand `opts_current$set` is used to fix it. `opts$current` is not something that would be used...

opts_current$set should error

FWIW @yihui it seems one of the usage is to be able to tweak some chunk options from within another hook https://github.com/wjschne/apaquarto/pull/17/files/a156dd63a7cf7ed27c07d3e6435ae2e092bed97a#diff-4f7262d8ecef665a4aecd5bd3e1767a34b6085bf40e2f8aa466edb19c9d693e4 Is there another design pattern to be able...