fix: Fix an issue accumulating aggregated metric values.

pywr / pywr-next

An experimental repository exploring ideas for a major revision to Pywr using Rust as a backend.

6 stars 4 forks source link

Closed jetuk closed 4 months ago

jetuk commented 4 months ago

The issue was caused by the lazy evaluation of .map() when collecting into an Option. This is now done explicitly.

Batch21 commented 4 months ago

The fix looks good but the PR appears to include many of the changes from #150. Was the idea to remove these?

jetuk commented 4 months ago

Yeah, not sure what I did wrong there. Fixed it now.