This patch is an alternative to D3486.
Currently bolt shooters have an attack range of 90 and a linear splash range of 9. This means units need to have a vision range of 100 to be able to see the bolt shooters that are attacking them. While this is already the case for some (e.g. heroes, elephants), it's not the case for most (e.g. infantry has 80).
This patch therefore gives all units (except animals and catafalques) a vision range of 100, which means all units can see all units that can see or attack them.
Details
Details
- Reviewers
- None
- Group Reviewers
Balancing
Check for completeness and correctness. Agree with the proposal.
Diff Detail
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- /ps/trunk
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 15457 Build 33831: Vulcan Build Jenkins Build 33830: Vulcan Build (macOS) Jenkins Build 33829: Vulcan Build (Windows) Jenkins Build 33828: arc lint + arc unit
Event Timeline
Comment Actions
Build is green
builderr-debug-macos.txt ld: warning: text-based stub file /System/Library/Frameworks//CoreAudio.framework/CoreAudio.tbd and library file /System/Library/Frameworks//CoreAudio.framework/CoreAudio are out of sync. Falling back to library file for linking. ld: warning: text-based stub file /System/Library/Frameworks//AudioToolbox.framework/AudioToolbox.tbd and library file /System/Library/Frameworks//AudioToolbox.framework/AudioToolbox are out of sync. Falling back to library file for linking. ld: warning: text-based stub file /System/Library/Frameworks//ForceFeedback.framework/ForceFeedback.tbd and library file /System/Library/Frameworks//ForceFeedback.framework/ForceFeedback are out of sync. Falling back to library file for linking. ld: warning: text-based stub file /System/Library/Frameworks//CoreVideo.framework/CoreVideo.tbd and library file /System/Library/Frameworks//CoreVideo.framework/CoreVideo are out of sync. Falling back to library file for linking. ld: warning: text-based stu
See https://jenkins.wildfiregames.com/job/macos-differential/3091/display/redirect for more details.
Comment Actions
Ah, you meant terrain elevation bonus. True.
I was confused because entity (i.e. footprint) height is irrelevant.
Comment Actions
Am rather strongly opposed to increasing vision ranges. The problem of artillery being invisible should be fixed by revealing the attacker temporarily.
Comment Actions
I am not in favor of increasing the vision range and also the same vision range for all units.
Comment Actions
From a gameability standpoint, I think what was done in 23b was fine. I also preferred vision tech.
From an accuracy (of combat representation) standpoint,
infantry are probably not likely to see what's being shot at them because they're probably too focused on not getting hit with melee weapons (ie., myopic focus).
Ranged attackers would be more likely to see distant targets because they (for the most part) don't need to worry about proximity threats. That and
ranged attackers typically find advantaged positions from which to fire (typically higher ground and advantaged aspects on enemy). So the vision range
advantage of ranged attackers I feel is adequately captured in the 23b representation of vision range.
If our goal is to create an iron age combat simulator, I think we have a lot of hack and slashing to do to make 0ad one of those. If our aim is to create
a gameable configuration, we should accept (we already do) some caricaturization of reality.