Skip to content
This repository was archived by the owner on Jul 1, 2022. It is now read-only.

Conversation

@xw220
Copy link
Contributor

@xw220 xw220 commented Jun 23, 2022

I watched 10 hours of diablo run by 4 fohdins. About 30% were failed games which aborted in middle run, especially on detecting that if the seal is openned. In almost every fail caused by this reason, the seal was openned successfully, but the decisions were closed for 5 times then the game was aborted.

By increasing the numbers of try and reduce the seal_opentemplates threshold to 0.7, the incorrect judgements on seal's open status seldom occur. Now the ratio of failed games typically reduced. It is about 12% mostly caused by temporary high latency or getting lost after teleport to pick item.

I watched 10 hours of diablo run by 4 fohdins. About 30% were fail games which aborted in middle run, especially on detecting that if the seal is openned.  In almost every fail game caused by this reason, the seal was openned successfully, but the decisions were closed for 5 times and the game was aborted. By increasing the numbers of try and reduce the seal_opentemplates threshold to 0.7, the incorrect judgements on seal's open status seldom occur, and the ratio of failed games typically reduced. Now it is about 12% mostly caused by temporary high latency or getting lost after teleport to pick item.
@Riddle1001
Copy link
Collaborator

@xw220 There are a couple other people working on chaos sanctuary too, if you'd like to talk to them join us on discord and @ one of the admins to get the contributor role. https://discord.gg/Jf3J8cuXWg

@jobithu
Copy link
Contributor

jobithu commented Jun 23, 2022

Interesting, because with the original parameters, the pressure tests revealed >95% success rate.

Most important point: do you have REDEMPTION bound to skills? typically templates for checking seals are obscured by corpses that are close by. redemption removes these to increase template matching.

If the reduction from 0.75 to 0.75 does not cause any false-postives (closed seals considered open) I wouldn't mind changing this value.

When it comes to number of tries: i had 7 when i started, then lowered to 5 to save time at runs. typically if you failed to open 5 times already, then something is really wrong, so adding another 2 or 3 tries most of the times wont change it and its better to accept that the run was a fail and keep going with a new one.

by the way: we are currently completely reworking diablo. you might be interested looking into that PR (-> #867). also, we consider to remove the template detection for seals and use hard-coded clicks to see if it is more efficient.

@xw220
Copy link
Contributor Author

xw220 commented Jun 23, 2022

Most important point: do you have REDEMPTION bound to skills? typically templates for checking seals are obscured by corpses that are close by. redemption removes these to increase template matching.

Yes, I bound redemption.

If the reduction from 0.75 to 0.75 does not cause any false-postives (closed seals considered open) I wouldn't mind changing this value.

Reduce to 0.7 do helps. I saw many 72% 73% in debug logs.

@jobithu jobithu merged commit 1ba5319 into aeon0:master Jun 23, 2022
@xw220 xw220 deleted the improve/diablo_sealdance branch June 24, 2022 13:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants