Skip to content

Fix latex repr of symbolic distributions #6231

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.

Sign up for GitHub

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 14 commits into from
Oct 30, 2022
Merged

Fix latex repr of symbolic distributions #6231

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
deb7764
Fix latex repr of nested variables
mattiadg Oct 19, 2022
8a07528
Adding option to include transformed variables in InferenceData (#6232)
dfm Oct 20, 2022
005818b
Time series clean up (#6240)
junpenglao Oct 20, 2022
e82404f
Remove inner $ symbols from latex representations
mattiadg Oct 20, 2022
1f35dcc
Remove inner $ symbols from latex representations
mattiadg Oct 20, 2022
a4723df
Add test for a full model that used to be rendered wrong
mattiadg Oct 21, 2022
8088049
Merge branch 'main' into fix_latex_print
mattiadg Oct 24, 2022
96836cc
Merge remote-tracking branch 'origin/main' into fix_latex_print
mattiadg Oct 25, 2022
f072d41
Fix failing test because of wrong expected value
mattiadg Oct 26, 2022
c306296
Use "\\operatorname{" instead of "operatorname" to determine if the l…
mattiadg Oct 26, 2022
7867066
Merge remote-tracking branch 'origin/main' into fix_latex_print
mattiadg Oct 26, 2022
8fe24c4
Add tests with the code mentioned in the issue
mattiadg Oct 26, 2022
cc3c21c
Merge branch 'main' into fix_latex_print
mattiadg Oct 30, 2022
1d5940d
Update tests to new distributions' notation
mattiadg Oct 30, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove inner $ symbols from latex representations 
* remove them from the string borders since they always represent the math environment
* fix tests to check the correct behavior
  • Loading branch information
@mattiadg
mattiadg committed Oct 21, 2022
commit 1f35dcce0df8d13e80057a4d77736dd12eb4243a
6 changes: 3 additions & 3 deletions pymc/tests/test_printing.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -126,11 +126,11 @@ def setup_class(self):
r"$\text{beta} \sim \operatorname{N}(0,~10)$",
r"$\text{Z} \sim \operatorname{N}(f(),~f())$",
r"$\text{nb_with_p_n} \sim \operatorname{NB}(10,~\text{nbp})$",
r"$\text{zip} \sim \operatorname{MarginalMixture}(f(),~\text{\$\operatorname{DiracDelta}(0)\$},~\text{\$\operatorname{Pois}(5)\$})$",
r"$\text{zip} \sim \operatorname{MarginalMixture}(f(),~\operatorname{DiracDelta}(0),~\operatorname{Pois}(5))$",
(
r"$\text{nested_mix} \sim \operatorname{MarginalMixture}(\text{<constant>},"
r"~\text{\$\operatorname{MarginalMixture}(f(),~\text{\\$\operatorname{DiracDelta}(0)\\$},~\text{\\$\operatorname{Pois}(5)\\$})\$},"
r"~\text{\$\operatorname{Censored}(\text{\\$\operatorname{Bern}(0.5)\\$},~-1,~1)\$})$"
r"~\operatorname{MarginalMixture}(f(),~\operatorname{DiracDelta}(0),~\operatorname{Pois}(5)),"
r"~\operatorname{Censored}(\operatorname{Bern}(0.5),~-1,~1))$"
),
r"$\text{Y_obs} \sim \operatorname{N}(\text{mu},~\text{sigma})$",
r"$\text{pot} \sim \operatorname{Potential}(f(\text{beta},~\text{alpha}))$",
Expand Down