Update {{ stage_name }}.qmd #33
Conversation
added descriptive comments to clear up the functionality for beginners with dso
| @@ -1,14 +1,22 @@ | |||
| ```{r} | |||
| library(conflicted) | |||
| library(dso) | |||
| #| label: load libraries | |||
There was a problem hiding this comment.
I think label should not contain spaces as it will be a hyperlink anchor, see https://quarto.org/docs/reference/cells/cells-knitr.html#attributes
Not sure we need labels at all here?
There was a problem hiding this comment.
label can contain spaces, but yes, we can remove spaces.
labels are good practice, as it shows progress in compilation, also it can be used for referencing block dependencies (spaces are there no problem). it would be good to have show good practice in the template
There was a problem hiding this comment.
Personally I dislike them because I forget to change them when I copy&paste a block and then compilation fails because of duplicate labels. But I agree in an ideal world one would use them.
Added descriptive comments to clear up the functionality of read_params and stage_here for beginners with dso.
It is important to mention the relation between
read_paramsanddvc.yamlfor beginners to clear up that parameters have to be specified there.