FlexMeasures / flexmeasures / 28902397985

Pull #1946

github

fix: count only device models, not stock entries, when enumerating devices in _prepare

The commodity_to_devices mapping and the num_flexible_devices count
were built from the re-copied flex_model, which includes stock-model
entries. This overran the sensors list (IndexError at sensor_d =
sensors[d]) whenever the flex-model contained a stock entry, and gave
stock entries bogus device indices in commodity groups. Enumerate
device_models instead, keeping the earlier num_flexible_devices =
len(device_models) and matching _compute_commodity_aggregate_schedules,
which already uses self._device_models.

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

465 of 503 new or added lines in 12 files covered. (92.45%)

14032 of 17317 relevant lines covered (81.03%)

0.81 hits per line