Nexela / PickerBeltTools

MIT License
2 stars 5 forks source link

Fix internal belt counter #5

Closed TheStaplergun closed 5 years ago

TheStaplergun commented 5 years ago
TheStaplergun commented 5 years ago

Currently the total belts comes out to inconsistent values even marking the same belt network from the same spot. This doesn't bode well in a deterministic game.

There's only four places in the code that belts get counted under the belts_read variable, and that's hidden behind data validation steps prior to the function it is embedded in being called.

TheStaplergun commented 5 years ago

Belt count is now consistent. Issue is with belt counter not matching actual total belts that are marked.

TheStaplergun commented 5 years ago

A rewrite is in progress.

TheStaplergun commented 5 years ago

Closing this issue because it's no longer relevant.