Skip to content

Increase timeout duration for bank interaction#165

Closed
Rico-Rodriguez wants to merge 1 commit intoWaspScripts:mainfrom
Rico-Rodriguez:patch-11
Closed

Increase timeout duration for bank interaction#165
Rico-Rodriguez wants to merge 1 commit intoWaspScripts:mainfrom
Rico-Rodriguez:patch-11

Conversation

@Rico-Rodriguez
Copy link
Contributor

This pull request makes a small adjustment to the bank interface logic, increasing the timeout duration to allow for more generous timing when waiting for a response.

  • Increased the timeout in the bank.simba interface by updating the RandomMode parameters from a range of 85–250 ms to 800–1500 ms, likely to improve reliability for slower systems or connections.

The quantity button was timing out before it had a chance to populate causing endless loops. You may want to modify these to be lower, but I found these values to work reliably 100% of the time.

@Torwent
Copy link
Collaborator

Torwent commented Feb 16, 2026

If you don't mind I want to check this in game first because this really shouldn't be needed.

Uptext is delayed by one frame, so even at 30 FPS which is the bare minimum I would recommend to use Simba that would mean you need 34ms, the minimum of 85ms should be more than enough.

If it's really required to increase this I don't mind doing it but I think it shouldn't really be increased more than double of the current values

@Rico-Rodriguez
Copy link
Contributor Author

@Torwent I was actually hopping back on to close this request. I agree, this can and should be handled by my script!

@Torwent
Copy link
Collaborator

Torwent commented Feb 16, 2026

I am not sure if it should be handled by your script, it'st just that this specific thing, theoretically, it shouldn't fail.
If it does, I'm just not sure the issue is the timeout, might just be OCR not working properly. Need to investigate!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants