Improve subterranean harvester pathfinding

[Starkku]

This PR should allow MovementZone=Subterannean harvesters to access docks that are in areas closed off from normal pathfinding such as cliffs and islands. Whether or not it fixes all issues with such harvesters is another thing.

[github-actions]

Nightly build for this pull request:

• compiled-dll-0003e84fff79c6612af6f507387ecda26af23b2b.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.

[CrimRecya]

It looks good. And I need to test whether it will be affected by the refinery on water surface.

[Coronia]

used Project Phantom as a test, but there're still some subterrenean miners getting stuck for unknown reason, mostly happened when they're trying to get to an ore field which is being harvested when they're on their ways

1 - stuck at ore field

2 - stuck at slope

[mevitar]

Did some games against the AI with this, and at some random point the game dropped to like 1 fps. This happened several times, but lasted only few seconds (except for one game, which didn't seem to fix itself so i had to quit). I also haven't encountered this in any build other than this one.

All these games where it happened were me and AI playing as sides with subterranean harvesters. I'll try investigating further if they are indeed the cause.

[mevitar]

No fps drops this time, but game crashed with this build, can't tell the reason from except.txt
https://drive.google.com/file/d/1xd3PoLCVYi60-u0V2jbiQue97VGWkg5t

EDIT: can be ignored, it happens because of airstrike flare, and AirstrikeTarget not being cleared up properly

[mevitar]

Haven't had fps drops anymore, but these games were played at like 40-45 fps. The fps drops happened only in games with 60 fps, so that might have been related to how fast the game was processing, or the drawer that was used.

Had only sporadic cases of miners getting stuck, only in the lategame when the resources are mined out on most of the map. If Ollerus didn't point them out i wouldn't even be paying attention.
I'm not sure how much of that was due to them being subterranean miners and some bug making them forget to mine, or simply stopping mining because they can't find anyting to mine anymore.

[Starkku]

Attempted to further improve the behaviour but I suspect it won't fix every issue particularly with harvesters getting stuck if there's too many on single ore patch or refinery.

• Further improvements to rally point handling, should now reliably travel to rally point first if one is set instead of jumping to a ore patch if there's one nearby.
• Applied similar idle mode exceptions to subterranean harvesters as teleporting ones have.

[Coronia]

when we were testing it in PP it gave us this, probably due to the recent include change

and it still didn't work quite well in MO after making miners move subterraneanly. They would stuck like this

[KuroNoSeiHai]

Hello. Excuse me, I found some other issues in subterranean harvester.

1. Phobos b47: If the subterranean harvester is ordered to move immediately after completing unloading (changing to the unload class), it may freeze indefinitely (visually stuck in the unload vxl). A workaround is to press the S key or issue multiple movement commands to resolve the freeze.
2. Vanilla Behavior: When manually ordering a subterranean harvester to return to the Ore Refinery, it unnecessarily circles around once and then finally stop. However, if using this nightly fix, the harvester returns to the refinery but does not stop completely—it resumes mining instead.

[mevitar]

But did you check the behavior in this build?
How harvesters behave in b44 is not indicative of how they behave here.

[KuroNoSeiHai]

But did you check the behavior in this build? How harvesters behave in b44 is not indicative of how they behave here.

I'm sorry, It's B47. I misspelled.

[CrimRecya]

Hello. Excuse me, I found some other issues in subterranean harvester.

1. Phobos b47: If the subterranean harvester is ordered to move immediately after completing unloading (changing to the unload class), it may freeze indefinitely (visually stuck in the unload vxl). A workaround is to press the S key or issue multiple movement commands to resolve the freeze.
2. Vanilla Behavior: When manually ordering a subterranean harvester to return to the Ore Refinery, it unnecessarily circles around once and then finally stop. However, if using this nightly fix, the harvester returns to the refinery but does not stop completely—it resumes mining instead.

The first issue starts to occur after b44, and b43 does not have this issue.
The second issue only occurs at close range (when the unit does not enter underground).