Skip to content
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

Crash on sim with no potion selected #87

Open
huntexD opened this issue Feb 1, 2022 · 6 comments
Open

Crash on sim with no potion selected #87

huntexD opened this issue Feb 1, 2022 · 6 comments
Labels
bug Something isn't working

Comments

@huntexD
Copy link

huntexD commented Feb 1, 2022

If you start a sim with no potion selected the sim will error and never finish.

Uncaught TypeError: Herblore.potions[this.player.potionID] is undefined
    computeFactor           sources/injectable/Consumables.js:344
    updateSingleResult      sources/injectable/Consumables.js:313
    update                  sources/injectable/Consumables.js:300
    performPostSimAnalysis  sources/injectable/Simulator.js:867
    startJob                sources/injectable/Simulator.js:943
    processWorkerMessage    sources/injectable/Simulator.js:994
    onmessage               sources/injectable/Simulator.js:541
Consumables.js:344:38
    computeFactor           sources/injectable/Consumables.js:344
    updateSingleResult      sources/injectable/Consumables.js:313
    update                  sources/injectable/Consumables.js:300
    performPostSimAnalysis  sources/injectable/Simulator.js:867
    startJob                sources/injectable/Simulator.js:943
    processWorkerMessage    sources/injectable/Simulator.js:994
    onmessage               sources/injectable/Simulator.js:541
@gmiclotte gmiclotte added the bug Something isn't working label Feb 1, 2022
@gmiclotte
Copy link
Collaborator

gmiclotte commented Feb 1, 2022

Can't reproduce this, got a settings export?

Thanks!

@huntexD
Copy link
Author

huntexD commented Feb 2, 2022

After having restarted my PC i can no longer reproduce this error either. Feel free to close this issue.

@yuriks
Copy link

yuriks commented Feb 13, 2022

I also experience this bug occasionally, though I haven't figured out a way to reproduce it reliably yet. EDIT: Forgot to mention, but the bug is temporarily resolved if you refresh the page to reload the game & simulator, until it starts happening again.

@catharsis71
Copy link

This has been happening for me sporadically for a while

It seems to happen when I load a gearset with a potion already enabled, and then I manually disable the potion in the simulator

image

Here I'm simulating the Fire God Dungeon only. I initially had Diamond Luck 4 enabled in-game and after doing some simulation as-is, I disabled it in the simulator to see what the difference would be, that's when the problem manifested

When the problem happens it hangs on "Cancel (5/6)". if I hit the cancel button it resets to 0, quickly counts to 5, then freezes again. After trying to cancel a few times the button just disables so I can't even try to cancel anymore.

After the problem manifests the simulator becomes unusable until the game is completely reloaded.

note that I usually crank up the trials and ticks so that might or might not be related

image

@catharsis71
Copy link

image

@gmiclotte
Copy link
Collaborator

Thanks for the detailed report, but I still can not reproduce this.

Apparently the sim thinks it consumed potions, while it clearly has not, since there is no active potion. I have no idea for the underlying cause, since I can not reproduce it.

However, I've now made a change that will prevent the crash and log a warning when it tries to process an "undefined potion", so it should at least no longer crash (with the next release).

Thanks for the reports!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants