FlexMeasures / flexmeasures / 29917718662
82%
main: 83%

LAST BUILD BRANCH: feat/1572-ui-commitments
DEFAULT BRANCH: main
Ran
Jobs 1
Files 171
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 82.426%. Remained the same
29917718662

Pull #2338

github

Flix6x 
fix(ui): render a sensor in every fast-chart subplot that configures it

A sensor configured in more than one asset graph row (sensors_to_show)
only rendered in the last row that listed it: groupData mapped each
sensor id to a single group, so later spec entries overwrote earlier
ones and every data row was routed to just that one subplot. The
Vega-Lite charts render such a sensor in each configured row.

Map each sensor id to the list of groups that configure it and route
every data row to all of them. Each group keeps its own series objects
(and thus its own points), and series colors are keyed by sensor name,
so the duplicated sensor keeps the same color in every subplot.

Found by @BelhsanHmida while testing #2312 (the bug is unrelated to
that PR's annotation work; it dates from the ECharts port in #2234).

Verified with a headless-Chrome harness driving renderFastChart with a
two-row spec sharing a sensor: before the fix the first row missed the
shared sensor's series; after it, both rows render it with identical
data and color, with no console errors.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HBYFTyXL1dLcqX6M76Av6e
Signed-off-by: F.N. Claessen <felix@seita.nl>
Pull Request #2338: Fix: render a sensor in every fast-chart subplot that configures it

15656 of 18994 relevant lines covered (82.43%)

0.82 hits per line

Jobs
ID Job ID Ran Files Coverage
1 29917718662.1 171
82.43
 GitHub Action Run
Source Files on build 29917718662