Skip to content
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

Minor Fix: Equation Formatting in Documentation #518

Open
wants to merge 1 commit into
base: release
Choose a base branch
from

Conversation

OleSeifert
Copy link

@OleSeifert OleSeifert commented Jan 31, 2025

Problem:
The equations in two files in the documentation do not render as LaTeX. This issue appears to stem from how GitHub markdown processes LaTeX equations (see [1]).

Solution:
I replaced the \( \) with $ $ and the \[ \] with $$ $$ as suggested in [1].

I encountered a minor issue in the docs/10_log-model_evaluation.md file:
It appears that GitHub's markdown LaTeX does not accept \_ within a \text{} macro. No matter how I tried, it would not render correctly on GitHub. I researched potential solutions but found nothing more effective (see [2]). I decided to rename it to \mathrm{mean\_deg}, which renders in a way that is clear on GitHub.

Additionally, my editor accidentally removed a small trailing whitespace.

 * Replace `\( \)` by `$ $` because the first one is not rendered in
   Markdown

 * Replace `\[ \]` by `$$ $$` due to the same reason

 * Rename `\text{mean\_degree}` to `\mathrm{mean\_deg}` because the
   original option would not render correctly on GitHub. After some
   research this seems to be the best fix.

 * Remove one trailing whitespace
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant