-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Planner gives inaccurate sythesis suggestions #481
Comments
cc @AlvISsReimu |
A simpler example. Import this and click calculate:
And the result shows that we have to synthesize a Because of the item id If tomorrow the issue still remains, I will dig into ArkPlanner further and see what causes the issue |
The problem seems not relate to the formulas. And I can't understand how ArkPlanner work. Giving up |
A correct example that reproduces the case: {"@type":"@penguin-statistics/planner/config","items":[{"id":"30115","need":1},{"id":"30084","need":4},{"id":"30033","have":5},{"id":"30023","have":4},{"id":"30042","have":10},{"id":"30053","have":3},{"id":"31014","have":4},{"id":"31013","have":5},{"id":"31024","have":3},{"id":"31023","have":5},{"id":"31033","have":5}],"options":{"byProduct":false,"requireExp":false,"requireLmb":false},"excludes":[]} This problem is due to the linear programming setting because the planner optimizes for minimized total sanity cost. Since synthesizing items cost no sanity, the planner will generate any result that meets the given requirements, allowing redundant synthesis operations. One possible solution is to add a small cost when synthesizing any item. I am trying to figure out whether there exists a better way to resolve this issue. |
@whzzt Can we compute the equivalent sanity cost from the synthesize LMB cost? Adding it to the synthesize cost shall do the trick. |
It is probably why no such problem exist when |
Environment
Mozilla/5.0 (X11; Linux x86_64; rv:95.0) Gecko/20100101 Firefox/95.0
URL
https://penguin-stats.io/planner
Steps to reproduce
Using the following planner configuration:
Select "Calculate" and view the "Syntheses" tab of the results panel.
What is expected?
If I've done my math right, the correct synthesis suggestions should be:
Polymerization Preparation
x1Manganese Trihydrate
x4Orirock Cube
x2Orirock Cluster
x2Orirock Concentration
x1What is actually happening?
After calculating, the "Syntheses" tab of the calculation result shows the following syntheses:
Polymerization Preparation
x1Manganese Trihydrate
x4Device
x1Integrated Device
x1Polyketon
x1Aketon
x1Oriron
x1Oriron Cluster
x1Oriron Block
x1Polyester Pack
x1Sugar
x1Sugar Pack
x1Orirock Cube
x2Orirock Cluster
x2Orirock Concentration
x1Incandescent Alloy Block
x1Polymerized Gel
x1Crystalline Circuit
x1Crystalline Electronic Unit
x1Many of these syntheses are unnecessary. For instance, there's obviously no reason to synthesize
Crystalline Electronic Unit
as neither the components nor product are involved at all in the needed materials. It looks like the results are suggesting synthesizing x1 of all materials not used in the expected syntheses.Note that "Consider by-products" is disabled, so the planner is not suggesting the extra syntheses for their byproducts.
I ran into this bug while using the planner normally. I've been able to reproduce it in different browsers on different platforms, so I don't think the problem's on my end.
This bug may be related to #407. However, that ticket was resolved as a non-issue because the described behavior was caused by the synthesis byproduct option. The behavior described here occurs whether synthesis byproducts are enabled or not, and seems to be a genuine bug.
I understand that the ArkPlanner is actually a separate service, but I figured I ought to report it here first through the penguin-stats bug reporter. If I should repost this issue to the ArkPlanner repo, let me know.
The text was updated successfully, but these errors were encountered: