Server UPS drops when many trains waiting about clusteriotrainteleports HOT 6 CLOSED

It is waiting 2 seconds to retry. Just don't have too many trains, problem solved. That's part of the logistic problem...

from clusteriotrainteleports.

I think the retry logic doesn't actually work correctly in some cases.

from clusteriotrainteleports.

from clusteriotrainteleports.

See #7

from clusteriotrainteleports.

if v.lastTry is nil, it will go in since it has not tried yet
if v.lastTry is not nil it will go to the next condition v.lastTry > event.tick - TELEPORT_COOLDOWN_TICKS, which would be true, if the v.lastTry was less than TELEPORT_COOLDOWN_TICKS ago and thus would not do anything this time around either.

If v.lastTry is nil, you don't want it to wait the 2 seconds, or it will take 2 seconds for each reroute to the next stop too.

from clusteriotrainteleports.

It appears that v.lastTry is not set if a station is found and all stations are blocked. A valid station is returned from find_station as long as one exists on the map even if it is blocked, so L793 should always be true. Maybe I misunderstand the code, but it seems like it will never sleep in this case.

from clusteriotrainteleports.