Closed rich-iannone closed 1 month ago
Attention: Patch coverage is 95.71429%
with 6 lines
in your changes missing coverage. Please review.
Project coverage is 86.49%. Comparing base (
23704ee
) to head (55d2fab
). Report is 8 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
great_tables/_helpers.py | 96.87% | 4 Missing :warning: |
great_tables/_formats.py | 80.00% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
From pairing, it sounds like this will be a helpful addition for the SciPy talk.
Let's...
define_units()
. e.g. it looks like there are 5 major modes.define_units()
are tested@machow I added a reference to define_units()
in the See Also section of the fmt_units()
docstring.
This PR adds the
fmt_units()
method. This performs a conversion of units in the units notation syntax (e.g.,"x10^9 / L"
or"x10^9 L^-1"
, etc.) to HTML. The method, like others of thefmt_*()
, only concerns itself with text transformations in the table body. Other methods will also gain the ability to convert text in units notation to nicely formatted HTML in later PRs.Here's an example that uses
fmt_units()
with theillness
dataset. It so happens that theunits
column has strings in units notation, so, we just need to point this method to that column:Fixes: https://github.com/posit-dev/great-tables/issues/211 Partially addresses the
.epic
issue: https://github.com/posit-dev/great-tables/issues/169