SEED-platform / seed / #9438

24%

Build Type

push

coveralls-python

Committed by web-flow

Commit Message

Stop embedding units in public feed values; add "<field>_units" sibling key (#5293)

* Stop embedding units in public feed values; add "<field>_units" sibling key

The public feed (feed.json/feed.html, via seed/utils/public.py) stringified
pint-aware values like site_eui into a single combined string, e.g.:

    "site_eui": "87.3 kilobritish_thermal_unit / foot ** 2 / year"

This meant consumers had to parse the unit back out of the value string
themselves. Now the value is left as a plain, unit-less number, and the units
are reported in a sibling "<field>_units" key instead:

    "site_eui": 87.3,
    "site_eui_units": "kilobritish_thermal_unit / foot ** 2 / year"

Added a test asserting a pint-aware field (gross_floor_area) comes back as a
plain number with its units in the new sibling key; confirmed it fails
against the old code and passes with the fix. Also verified live against a
real org's data via the running dev server.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Coverage Stats

0 of 8 new or added lines in 2 files covered. (0.0%)

2 existing lines in 1 file now uncovered.

12434 of 50722 relevant lines covered (24.51%)

0.25 hits per line