Phobos icon indicating copy to clipboard operation
Phobos copied to clipboard

Refactor the FogOfWar

Open secsome opened this issue 4 years ago • 5 comments

resolves #28

secsome avatar Feb 18 '22 13:02 secsome

Nightly build for this pull request:

github-actions[bot] avatar Feb 18 '22 13:02 github-actions[bot]

Looking Good!

My first comment is ive noticed quite a few issues when drawing overlay (especially bridges) from underneath Shroud/Fog I also saw some weird ore/gem flickering, but it wasnt as repeatable so i have no evidence image

Animations such as smoke appear underneath the fog

If you look closely into the shroud, you can see the terrain behind it to an extent However, in my image the heights of the terrain are 4-8, and are very dim image

In other maps which have lower ground on 0, you can barely see anything under the shroud image

Not sure if this corresponds with map lighting, but either way it is fairly inconsistent

CatTanker avatar Feb 18 '22 19:02 CatTanker

Here are my findings:

  1. SpySat=yes does not work and fog remains intact.
  2. Engineers can be ordered to enter structures (tech structures) through fog, but when they get to the structure they just stand still.
  3. Units attacking my building were concealed by fog, after they destroyed the building they also got concealed by shroud, guessing this is to do with RevealOnFire on weapons - since it defaults to yes: image
  4. I am able to see laserlinetrails through fog (projectiles and units): image
  5. Superweapons with RevealToAll show up but are not visible, if that makes sense: image
  6. Got a Exception code: C0000005 at 2D8854E0, uploaded file to #phobos-chat
  7. I think a SW.FireIntoFog tag needs to be implemented as this is such a prominent feature, SW.FireIntoShroud is completely ignored through fog, and if you have a Generals style mod where there is no shroud, only fog, this tag becomes useless

ayylmaoRotE avatar Mar 03 '22 13:03 ayylmaoRotE

Played online with Cat today, no desyncs or FoW related crashes. Same issues as above still apply.

Another thing we noticed is that you can place buildings in fog provided enough adjacency, should probably respect fog as it does shroud.

ayylmaoRotE avatar Apr 03 '22 08:04 ayylmaoRotE

To add, performance wise it was fine On an average/standard connection/tunnel, we achieved around 43fps on average, aka no noticeable performance drops in comparison to normal gameplay.

CatTanker avatar Apr 03 '22 09:04 CatTanker