Skip to content

Replace MathJax with KaTeX. #1362

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

Merged
merged 3 commits into from
Apr 20, 2021
Merged

Replace MathJax with KaTeX. #1362

merged 3 commits into from
Apr 20, 2021

Conversation

sbrodehl
Copy link
Contributor

This pull request replaces MathJax with KaTeX.

Multiple issues with formulae displayed with MathJax have come up in the past and partly have been solved, but the current state is still not satisfying.

PyTorch Docs uses KaTeX (for years now) without any major problems so it seems only natural to replace MathJax with KaTeX in the PyTorch Tutorials as well!

These are related issues and pull requests:

This change not only streamlines the visual appearance on the website, but (IMHO) it also looks way better ;-)

Here is one example which shows the benefits of KaTeX:

  • MathJax
    MathJax-autograd_tutorial html_optional-reading-vector-calculus-using-autograd

  • KaTeX
    KaTeX-autograd_tutorial html_optional-reading-vector-calculus-using-autograd

@netlify
Copy link

netlify bot commented Feb 17, 2021

Deploy preview for pytorch-tutorials-preview ready!

Built with commit b336092

https://deploy-preview-1362--pytorch-tutorials-preview.netlify.app

@sbrodehl sbrodehl changed the title Replace MathJax with Katex. Replace MathJax with KaTeX. Feb 17, 2021
@netlify
Copy link

netlify bot commented Feb 17, 2021

Deploy preview for pytorch-tutorials-preview ready!

Built with commit 9f4e480

https://deploy-preview-1362--pytorch-tutorials-preview.netlify.app

@holly1238 holly1238 merged commit 35ae0ea into pytorch:master Apr 20, 2021
@sbrodehl sbrodehl deleted the dev-katex branch April 20, 2021 16:38
rodrigo-techera pushed a commit to Experience-Monks/tutorials that referenced this pull request Nov 29, 2021
Co-authored-by: holly1238 <77758406+holly1238@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants