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

LaTeX not rendering in boundary_conditions.md #3905

Closed
jbisits opened this issue Nov 6, 2024 · 1 comment · Fixed by #3906
Closed

LaTeX not rendering in boundary_conditions.md #3905

jbisits opened this issue Nov 6, 2024 · 1 comment · Fixed by #3906
Labels
documentation 📜 The sacred scrolls

Comments

@jbisits
Copy link
Contributor

jbisits commented Nov 6, 2024

The last equation in the open boundary conditions documentation does not display. I think this is due to wrapping begin{align} ... \end{align} in \begin{equation} ... \end{equation}

```math
\begin{equation}
\begin{align}
u^{n+1}_{1jk} &\approx u^{n+1}_{3jk} + (u^{n+1}_{2jk} - u^{n+1}_{jk4}) / 2 + \mathcal{O}(\Delta x^2),\\
&= u^\star_{1jk} - \Delta t \left(\boldsymbol{\nabla}p^{n+1}_{3jk} + (\boldsymbol{\nabla}p^{n+1}_{2jk} - \boldsymbol{\nabla}p^{n+1}_{4jk}) / 2\right) + \mathcal{O}(\Delta x^2),\\
&\approx u^\star_{1jk} + \mathcal{O}(\Delta x^2) + \mathcal{O}(\Delta t).
\end{align}
\end{equation}
```

Can either be fixed by align -> aligned or removing \begin{equation} ... \end{equation} (both options correctly render the equation in the markdown preview on github). I have opened a PR to make this change.

@glwagner
Copy link
Member

glwagner commented Nov 6, 2024

thank you!

@navidcy navidcy added the documentation 📜 The sacred scrolls label Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation 📜 The sacred scrolls
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants