Question on handling Animation Checking

[grape303 1]

I have a mining script that is running OK, but could easily improve performance with 1 fix.
My issue is:
   when I'm mining an ore that is going to take a bit, the animation will revert back to -1 after a few seconds of failed mining rolls & then it will continue mining.
   During this half second, give or take a few, the window of no animation (-1) will cause my script to attempt to mine again.
This is an issue because it resets the chance for a successful mine. Making it extremely inefficient. 

 

Any assistance would be greatly appreciated. 
Thank you!

[camelCase 240]

Sleep.sleepUntil(Inventory::isFull, Players.getLocal()::isAnimating, 1200, 100)

sleeps until your inventory is full, or 1200ms of not animating, checking if you are animating every 100ms

the 2nd condition resets the timer when true

[grape303 1]

@camalCaseThis is definitely a major improvement of what I had put, so thank you. Improvement being 1-3 swing attempts vs 4-5.

unfortunately, There are still occurrences of of multiple "you swing your pick at the rock" for the very very short -1 animation. I may have to get creative with inventory check & message. Do you think that could be a viable route worth exploring?

 

[MaximusPrimo 64]

No need. That snippet should work fine. Change the 1200 timeout to a higher number like 2500 and see if that helps 

[grape303 1]

I believe it is working good enough for me to move onto wrapping up paints, cleaning up code, and submitting review!
Thank you both