Investment appears to decommission too much in this case #1131
Description
Describe the bug
In a version of the missing_commodity model, where there are divisible assets, the algorithm appear to decommission too much before the 2nd MSY dispatch, and then that dispatch fails.
This only seems to happen when both GASPRD and GASPRC processes have the unit_size parameter defined. Not when only one of them has it. Odd!
To Reproduce
Run the attached model. See:
[11:14:03 INFO muse2::simulation] Milestone year: 2020
[11:14:03 INFO muse2::simulation] Running dispatch optimisation...
[11:14:03 INFO muse2::simulation] Milestone year: 2021
[11:14:03 INFO muse2::simulation] Running agent investment...
[11:14:03 INFO muse2::simulation] Running dispatch optimisation...
[11:14:03 INFO muse2::simulation] Running agent investment...
[11:14:04 INFO muse2::simulation] Running dispatch optimisation...
[11:14:04 INFO muse2::simulation] Prices converged after 2 iterations
[11:14:04 INFO muse2::simulation] Running final dispatch optimisation for year 2021...
[11:14:04 ERROR muse2] The solver has indicated that the problem is infeasible, probably because the supplied assets could not meet the required demand. Demand was not met for the following markets: [(CommodityID("ELCTRI"), RegionID("GBR"))]
Expected behavior
I think this should not fail. Algorithm should decommission one unit less, and then I think it should be OK.