Follows D3209
One can now use the 'very obstructed' message, which indicates a unit is quite stuck. The 'relaxed range check' can use this to try harder to reach the target point.
I still relax on 'obstructed', which comes much faster, if there are additional orders after, as that improves behaviour.
Details
- Reviewers
- None
Order 30 units to move to a single point (not in formation). Compare.
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- temp
- Lint
Lint OK Severity Location Code Message Warning binaries/data/mods/public/simulation/components/UnitAI.js:342 ESLintBear (consistent-return) ESLintBear (consistent-return) Warning binaries/data/mods/public/simulation/components/UnitAI.js:785 ESLintBear (no-else-return) ESLintBear (no-else-return) Warning binaries/data/mods/public/simulation/components/UnitAI.js:2148 ESLintBear (operator-linebreak) ESLintBear (operator-linebreak) - Unit
No Unit Test Coverage - Build Status
Buildable 14997 Build 32400: Vulcan Build Jenkins Build 32399: Vulcan Build (macOS) Build 32398: Vulcan Build (Windows) Jenkins Build 32397: arc lint + arc unit
Event Timeline
Build failure - The Moirai have given mortals hearts that can endure.
Link to build: https://jenkins.wildfiregames.com/job/macos-differential/2646/display/redirect
Build failure - The Moirai have given mortals hearts that can endure.
Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/3738/display/redirect
Build failure - The Moirai have given mortals hearts that can endure.
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/4309/display/redirect
Rebased. Planning to merge still as the current behaviour can look buggy, even if it's not really problematic.
Although I have to say that I do not see the need and it merely adds pathfinder strain. IMHO.
If you take a bunch of units and order them to move (not in formation) to a point, they can occasionally stop 8 meters away (or not even move at all). This looks odd.
I think I'll still want this, but probably not before Unit Pushing is merged / might be irrelevant then.