Skip to content
6 changes: 4 additions & 2 deletions CHANGES.rst
Original file line number Diff line number Diff line change
Expand Up @@ -62,8 +62,10 @@ Bugfixes
durations to the total number of seconds (with fractional part). This is done
by the new transformer :class:`DurationToFloat`. :pr:`2069` by
:user:`Riccardo Cappuzzo <rcap107>`.


- An error that could arise when running ``TableReport`` on dataframes containing
double dollar (``$$``) signs has been fixed.
:pr:`2154` by :user:`Katerina Michenina <Michenina-Lab>`,
:user:`CecilyTS <CecilyTS>`, :user:`Eve Rabin <eve2705>`.

Deprecations
------------
Expand Down
1 change: 1 addition & 0 deletions skrub/_reporting/_plotting.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@
"boxplot.flierprops.color": _TEXT_COLOR_PLACEHOLDER,
"boxplot.flierprops.markeredgecolor": _TEXT_COLOR_PLACEHOLDER,
"boxplot.whiskerprops.color": _TEXT_COLOR_PLACEHOLDER,
"text.parse_math": False,
}


Expand Down
14 changes: 14 additions & 0 deletions skrub/_reporting/tests/test_summarize.py
Original file line number Diff line number Diff line change
Expand Up @@ -309,3 +309,17 @@ def test_with_associations_and_seed(monkeypatch, air_quality):
"Computing summary with different seeds should produce different "
"associations."
)


def test_dollar_sign(df_module):
"""
non-regression test for

https://github.com/skrub-data/skrub/issues/2097

without the right rcparams matplotlib interprets the $ as latex and crashes.
"""
df = df_module.make_dataframe(
{"c": ["something with $$", "something else so column is not constant"]}
)
summarize_dataframe(df, with_plots=True, verbose=0)