Differential D966
Don't show silhouettes through fields and berries Authored by temple on Oct 21 2017, 3:04 AM.
Details We use silhouettes for units hiding behind trees and buildings. We can walk over fields and berries instead of hiding behind them, so the silhouettes are more distracting than helpful.   This patch changes the field and berries to be like the grass actors on The Nile, for example:  Agree with the change. (Berry bushes still need to be cleaned up in D845.)
Diff Detail
Event Timeline
Comment Actions Use berry template instead of bush, since we now have wood bushes (that can't be walked over so should still show silhouettes). Comment Actions butbutbut fields and bushes can hide whole chicken f.e. so in combination with D965 this would kinda break Comment Actions That's a good point, so I would prefer a more proper fix to #4829. On the other hand, in a22 fields and bushes can hide whole chickens, so we're not making anything worse. Comment Actions I wonder if it would be nice to use the footprint.height tag for this, as in when the height of the occluder is heigher than the occluded => show the silhouette when the occluded template wants that, otherwise let it depend also on the occluder visualactor stuff. Comment Actions I think this is the wrong way to go about it. Instead we should render silhouette casters as anti-occluders using the depth map, so that a silhouette is never renderer on top of the actual visible model, which just looks weird. Comment Actions It'll be up to someone else to try that because I don't have much of an idea about what's going on. | ||||||||||