-
-
Notifications
You must be signed in to change notification settings - Fork 457
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
Upgrade Julia from 1.6 to 1.10 #2000
Conversation
Once the build has completed, you can preview your PR at this URL: https://julialang.netlify.app/previews/PR2000/ in ~15 minutes |
I think you want to either use julia v1.6 locally, or update
|
Which is what just happened: https://github.com/JuliaLang/www.julialang.org/actions/runs/7371299165/job/20058535468?pr=2000#step:10:25 |
Seems like the setup-julia action reads
|
Co-authored-by: Mosè Giordano <[email protected]>
Ok, that specific error is still happening now after upgrading to Julia v1.10.0, but using Julia v1.6 with a manifest for Julia v1.10 caused loads of warnings during precompilation: https://github.com/JuliaLang/www.julialang.org/actions/runs/7371299165/job/20058535468?pr=2000#step:9:23. I guess updating some of the packages in the environment screwed up building one of the pages. |
Strange. When I run |
The issue is that Franklin doesn't know about the math language and so, in old versions, renders
as generic text while in new versions it gives a warning which we then identify and escalate to an error. |
Removing this tag should be a no-op from a served-html perspective. This should be verified by comparing |
The page https://julialang.netlify.app/previews/pr2000/blog/ renders poorly now. Most blog posts are untitled. |
@tlienart Can we request you to take a look here? |
a5b3f3b
to
7a58d4b
Compare
069a438
to
cd283bc
Compare
@LilithHafner I hope you don't mind I took the liberty to update and figure out how to fix this. The blog list now renders properly. See the comment. Good to merge? |
Co-authored-by: Mosè Giordano <[email protected]>
@@ -618,7 +618,7 @@ Instead, what we are learning is the tiny ODE system from which the ODE | |||
solution is generated. I.e., the neural network inside the neural_ode | |||
layer learns this function: | |||
|
|||
```math | |||
```text |
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.
what happened to tex support?
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.
Bump. I'll merge later today otherwise. Looks good to me. |
When running locally I got a bunch of "closewrite" not defined errors and the website didn't renter properly. After running
Everything worked fine. This PR commits the changes produced by running that line.
Before:
After: