Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • dumux dumux
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 101
    • Issues 101
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 91
    • Merge requests 91
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • dumux-repositoriesdumux-repositories
  • dumuxdumux
  • Merge requests
  • !2143

Cleanup/use mathjax

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Simon Emmert requested to merge cleanup/use-mathjax into master May 27, 2020
  • Overview 16
  • Commits 2
  • Pipelines 0
  • Changes 14

What this MR does / why does DuMux need it: Use MathJax to render equations for doxygen. I also replaced \textup with \text for now

I do not see a problem with \textbf and \bf as they are listed as supported by mathjax (http://docs.mathjax.org/en/latest/input/tex/macros/index.html) But mabye this is also related to browser versions.

Runtime reduces to 1m45.409s on my machine (without tests already), compared to 3m58.628s (without tests but with images) or 8m29.095s (with all tests and images)

The only downside I see when using MathJax is that MathJax actually takes some time to render once I click on a page.

Edit: The goal was to make formulas look nicer in doxygen.

  • Add doc/doxygen/Doxylocal +USE_MATHJAX = YES
  • Get rid of \textup, and \mbox{\bf} as they are not properly parsed by mathjax
  • Further commands that are not parsed properly were not found

\textbf stays for now.

Which issue this MR fixes: fixes #882 (closed)

Special notes for your reviewer: Needs to be merged after !2141 (merged)

Edited May 27, 2020 by Simon Emmert
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: cleanup/use-mathjax