-
Notifications
You must be signed in to change notification settings - Fork 55
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
Move tutorials to Quarto #638
Conversation
Codecov Report
@@ Coverage Diff @@
## master #638 +/- ##
==========================================
- Coverage 99.20% 99.14% -0.06%
==========================================
Files 106 106
Lines 10215 10210 -5
==========================================
- Hits 10134 10123 -11
- Misses 81 87 +6 see 8 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Hm, GLMakie.jl doesn't work on CI. @kellertuer , what do you think about rendering images locally and uploading them? How should I do that? |
I think there are two possibilities, either pretender just the one image and load it in the file or I would prefer the second I think, and do the same with two tutorials in Manopt.jl as well, you just have to exclude them then here in the You can also see that both markdown files and some images are actually committed https://github.com/JuliaManifolds/Manopt.jl/tree/master/docs/src/tutorials For the regression example the reason is the same as yours, we do not have Asymptote on CI, the other one just runs too long on CI. |
I went with the first option, it looks a bit easier. |
I will try to find time to review this tomorrow. At first glance: You created two quarto notebooks but only deleted one old jl file. What about the other one? |
I haven't deleted the "working in charts" .jl file because it still has some interactivity parts that couldn't be ported to the Quarto version. |
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.
Here are a few comments, overall – nice to have them in the new format. I hope we get a few more somewhen (maybe one on metrics, one on Lie,...)
Co-authored-by: Ronny Bergmann <[email protected]>
Yes, Quarto looks nice. I think I will keep using it for new tutorials. |
In VS Code there is also a live html preview (Live as in re-render per save action, not as live as Pluto). Then writing these tutorials is very nice – and I feel a bit better fitting than Pluto even. |
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.
(given that tests pass)
Thanks for moving these :)
I've moved hand gesture tutorial to Quarto, I think it makes it easier to discover. And Quarto is nice.
I'm thinking about moving the chart/atlas tutorials but then we would lose all interactivity.