FlexMeasures / flexmeasures / 10962858010

74%

Build Type

push

github

Committed by web-flow

Commit Message

Belief chart: legends possible per plot (#1176)

* field definitions inside sensors_to_show loop, do not list all sensors on the overall chart

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* add combine_legend parameter to control if plots have their own legend

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* actually pass parameter through correctly (was still in testing mode)

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* feat: expose the combine_legends parameter (and rename it to suggest an instruction rather than a property)

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: stop creating a fixed spacing between vertically concatenated charts

Signed-off-by: F.N. Claessen <felix@seita.nl>

* style(UI): move color legend to the right

Signed-off-by: F.N. Claessen <felix@seita.nl>

* dev: for now, add an explicit JS variable to toggle the combined legend on and off (so for testing, you don't need to restart the server)

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: keep old legend position if combining legends

Signed-off-by: F.N. Claessen <felix@seita.nl>

* style: black

Signed-off-by: F.N. Claessen <felix@seita.nl>

* add changelog

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

---------

Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: F.N. Claessen <felix@seita.nl>

Run Details

6255 of 10135 branches covered (61.72%)

Branch coverage included in aggregate %.

0 of 5 new or added lines in 1 file covered. (0.0%)

2 existing lines in 1 file now uncovered.

7888 of 10891 relevant lines covered (72.43%)

5.74 hits per line