Closed twentylemon closed 2 months ago
Attention: Patch coverage is 79.06977%
with 9 lines
in your changes missing coverage. Please review.
Project coverage is 95.99%. Comparing base (
33a2a26
) to head (bd868a6
). Report is 1 commits behind head on main.
Files with missing lines | Patch % | Lines |
---|---|---|
duckbot/cogs/games/satisfy/solver.py | 79.06% | 0 Missing and 9 partials :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Summary
Fixes https://github.com/duck-dynasty/duckbot/issues/958. The objective function is mostly unchanged by looking at it, but it should consider iron plates to be 1.5x more costly than iron ore, for example; instead of merely counting the rate of items produced. I wasn't able to think of a test case to show the theoretical bad behaviour. I may add some test cases in the future, or as we run into weird solves.
also fixed a bug in a recipe. #python.
Checklist
format
in the repository rootpytest
in the repository root