-
Notifications
You must be signed in to change notification settings - Fork 980
Expand docs for julia engine
#1573
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Expand docs for julia engine
#1573
Conversation
|
@jkrumbiegel any other features you can think of that would be worth mentioning here? |
jkrumbiegel
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me otherwise, I can't think of anything big that's missing
38b0d91 to
f679efc
Compare
|
/deploy-preview |
|
🚀 Deployed on https://deploy-preview-1573.quarto.org |
docs/computations/julia.qmd
Outdated
| is using, since the global environment is not included in the load path | ||
| provided to Julia, unlike the behaviour of a Julia REPL session. | ||
| ### Caching |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now that julia engine support caching, a link to this part from this part could be useful: https://quarto.org/docs/projects/code-execution.html#cache
BTW do --cache-refresh works with Julia engine ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BTW do --cache-refresh works with Julia engine ?
Probably not currently. Do you happen to know if CLI flags get forwarded to the julia process along with all the frontmatter metadata from the notebook? If it does then we could probably support that behaviour.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably format.execute.cache: "refresh" is the correct field in the payload? If that's the case then, yes, we could support it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have quickly look
--cache-refresh does set internal variable
https://github.com/quarto-dev/quarto-cli/blob/0502c32d72bbc90cdd4736135bd7f645e773ff37/src/command/render/flags.ts#L272-L275
that is use to modify execute.cache
https://github.com/quarto-dev/quarto-cli/blob/0502c32d72bbc90cdd4736135bd7f645e773ff37/src/command/render/render-contexts.ts#L190-L193
so basically this is same as
execute:
cache: refresh
So you have access to it already same way as Jupyter
https://github.com/quarto-dev/quarto-cli/blob/0502c32d72bbc90cdd4736135bd7f645e773ff37/src/resources/jupyter/notebook.py#L116-L139
Does it help ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, that does help, thanks.
|
I think a "highlight" is the right way to expose these changes. I've added one. |
cwickham
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few wording suggestions.
Co-authored-by: Charlotte Wickham <cwickham@gmail.com>
|
Anything else needed in this PR or is this able to be merged? |
|
/deploy-preview |
|
🚀 Deployed on https://deploy-preview-1573.quarto.org |
cwickham
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
As requested in quarto-dev/quarto-cli#12121 (comment). Adds some additional docs to the section covering the
juliaengine that highlights some or the main features it currently has.