Skip to content

Conversation

@sayalaruano
Copy link
Collaborator

No description provided.

@sayalaruano sayalaruano requested a review from enryH March 25, 2025 12:38
Copy link
Collaborator

@enryH enryH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's good. You could add the specification of the output directory to argument parser which is already in get_report.

@sayalaruano sayalaruano merged commit 715081b into main Mar 26, 2025
16 checks passed
@sayalaruano sayalaruano deleted the quarto-checks branch March 26, 2025 07:06
cd docs
vuegen -dir example_data/Earth_microbiome_vuegen_demo_notebook -rt pdf -qt_checks
vuegen -c example_data/Earth_microbiome_vuegen_demo_notebook/Earth_microbiome_vuegen_demo_notebook_config.yaml -rt pdf
vuegen -c example_data/Earth_microbiome_vuegen_demo_notebook/Earth_microbiome_vuegen_demo_notebook_config.yaml -rt pdf -qt_checks
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so here it is needed to install tinytex on the fly, right? but just not in the docker image?

It's good that this has to be activated, so that updates do not just happen. but we should document this as well.

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

Successfully merging this pull request may close these issues.

3 participants