Tablist placeholder refreshing about powerboard HOT 8 OPEN

That's correct, it doesn't need to check if it is lower than 9999.
But if the speed is below 0, it should be just ignored because everything below 0 is a static line which does not need to be updated and therefore the tablist would update each tick unnecessarily if the interval is set to 1.

from powerboard.

Sadly there seems to be no easy fix for this.
One way to reduce the unnecessary calls would be to split header and footer updates.
A better solution might be to cache the lines with the text already replaced with placeholders so it only needs to update/replace the placeholders that are actually needed.

from powerboard.

from powerboard.

Thanks @NicoNekoDev
This will not exactly fix the problem which is described in this issue (You probably didn't understand it correctly) but it will actually fix another problem that I didn't even think about.

from powerboard.

Okay, I was wrong. The actual fix it would be to set the interval to 1 if is lower than 1. Yeah, it fixes the refreshes not working on tablist.

from powerboard.

The actual fix it would be to set the interval to 1 if is lower than 1

No, the code that you've poseted is correct, because -1 is meant to disabling refreshing for that line

tablist.yml description:

If you have static scores (no animations or updates needed): Set the 'speed' value to '-1' or '9999'. Then the scheduler won't start to save performance.

from powerboard.

Yes, my code is correct, but after a bit of review, I came to conclusion that doing if (speed < 1) interval = 1 because:

1. it doesn't go higher than the set interval (20 * 60)
2. it only requires to not go bellow 1, not below or equal to 0

from powerboard.

However, I have changed the code and just made the startXXAnimation functions a boolean which returns if the scheduler has been started or not (It does not start, if the speed is below 0 or higher than 9999). If it has started it executes Math.min, otherwise it does not.
051bb83

from powerboard.