Miners back to work when ore regenerated

[TaranDahl]

• Now miners will no longer withdraw from the Harvest mission due to mineral depletion and will periodically attempt to return to work.

[github-actions]

Nightly build for this pull request:

• compiled-dll-96efc19fac3d325cce3c2d30c3c48b9b649013b1.zip
  These artifacts will expire in 90 days and will not be available for download after that time.

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

[CCCP84]

I checked it works, the harvesters come to life one after the other, not at the same time, it looks very good

[CCCP84]

I think this should be made permanently enabled and not so that players have to enable it in the map or something else. no one will want to disable this option. there is no reason for this

[CCCP84]

Besides, it seems to solve this problem too:

CnCNet/cncnet-yr-client-package#303

in general this is a very significant improvement over the original game. There's always been a problem with these harvesters. In this regard, everyone has already gotten used to living with it. now the convenience will be significantly increased

[TaranDahl]

I think this should be made permanently enabled

Fine by me. What do you think? @Metadorius

[CCCP84]

How does it work? Is there some kind of timer that checks for ore at certain times? If so, you can make the delay much longer because checking more frequently is unnecessary and could cause lags with a lot of harvesters. Or no? I think checking every one or 1.5 minutes would be fine.
Although apparently there is no timer here, right?

[TaranDahl]

How does it work? Is there some kind of timer that checks for ore at certain times?

Yeah. The mission timer is used for the interval.

If so, you can make the delay much longer because checking more frequently is unnecessary and could cause lags with a lot of harvesters. Or no?

I used a hardcoded value, 100 frames. Regarding whether it will cause lag, I think there's no need to worry. This check cycle is longer than the original check cycle of the "miner returns to work" behavior (which is about 1 second). If the original CMIN didn't cause lag, then the new behavior won't cause lag either.

[Metadorius]

Fine by me. What do you think? @Metadorius

I agree

[TaranDahl]

Fucked by git