CyclopsMC / IntegratedCrafting / 13226575033

65%

master-1.12: 27%

push

github

Fix over-estimation of storage contents when calculating jobs

This could occur when an ingredient could be partially extracted from
storage and partially had to be autocrafted via a sub-job.
The simulated extraction memory was being set to an amount that was too
low, which caused the algorithm to incorrectly think there was more to
extract.
The flaw in reasoning before this commit was that the simulation
extraction memory would only increment. But this is false, since it will
decrement when sub-jobs are calculated, and this decrement was not taken
into account.

Closes #125

695 of 1248 branches covered (55.69%)

Branch coverage included in aggregate %.

2136 of 3244 relevant lines covered (65.84%)

3.19 hits per line