Page MenuHomeWildfire Games

[WIP] Reveal attackers in FoW/SoD
Needs ReviewPublic

Authored by wraitii on Thu, Jan 28, 12:16 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

The intent of this patch is to reveal attackers when they are in FOW/SOD, to avoid situations where attackers outrange their target.

I believe it works from the simulation POV, however the shaders make it buggy: because LOS is a texture, the units in SOD are rendered black-on-black, and in SOD are rendered 'grayed out' (but you can see it works).
I need to find a fix for that, obviously.

Test Plan

Get attacked form SOD/FOW (women work well because of low LOS)

(you can run the bundled combat demo)

Event Timeline

wraitii created this revision.Thu, Jan 28, 12:16 PM
Owners added a subscriber: Restricted Owners Package.Thu, Jan 28, 12:16 PM
wraitii edited the test plan for this revision. (Show Details)Thu, Jan 28, 12:17 PM
wraitii edited the summary of this revision. (Show Details)Thu, Jan 28, 12:19 PM

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/3101/display/redirect for more details.

wraitii requested review of this revision.Thu, Jan 28, 12:24 PM
wraitii added inline comments.Thu, Jan 28, 12:27 PM
binaries/data/mods/public/simulation/components/Visibility.js
120

This should totally be in Vision :p