fix LaTeX dot in diversification ratio formula#1788
fix LaTeX dot in diversification ratio formula#1788saishreyakumar wants to merge 5 commits intopymc-labs:mainfrom
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1788 +/- ##
=======================================
Coverage 92.07% 92.07%
=======================================
Files 62 62
Lines 7398 7398
=======================================
Hits 6812 6812
Misses 586 586 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Thanks for the edits @saishreyakumar I'm see some issues with it still |
|
Thank you for the PR @saishreyakumar ! Do you need a hand finishing this up? |
|
|
||
| .. math:: | ||
| DR = \frac{\\sum_{i=1}^{n} w_i \\cdot \\sigma_i}{\\sigma_p} | ||
| DR = \frac{\sum_{i=1}^{n} w_i \cdot \sigma_i}{\sigma_p} \ |
There was a problem hiding this comment.
According to the Sphinx documentation, the equation doesn't need a newline after the math call:
When the math is only one line of text, it can also be given as a directive argument:
.. math:: (a + b)^2 = a^2 + 2ab + b^2
Also, maybe the line should not terminate with \ which might be the cause of the rendering issue @williambdean is seeing..
Description
Replaced \cdot with . in the LaTeX formula for the diversification ratio in the docstring to fix the rendering issue.
Related Issue
Checklist
pre-commit.ci autofixto auto-fix.📚 Documentation preview 📚: https://pymc-marketing--1788.org.readthedocs.build/en/1788/