Per uploaded file sensorvis1.diff (8.6 KiB)

Per uploaded file sensorvis2.diff (9.2 KiB)

Fix bug that left visibility remain in areas where units had died.

cybersphinx commented

Interesting. I didn't see any obvious problems, subjective performamce was a bit lower.

Zarel changed type from defect to enhancement

Zarel commented

cough

Per uploaded file sensorvis3.diff (10.1 KiB)

Added support for allied vision.

Per uploaded file sensorvis4.diff (10.8 KiB)

Also show fog of war in radar minimap. Add some more helpful comments to code.

Per changed status from new to closed

Per set resolution to fixed

Per commented

(In [8900]) Add map and radar minimap display of true fog of war to the game. This is the same
as what you see when you previously held in the 'z' button. That is, you see where
you have sensor coverage. Implemented using reference counting on each tile that is
unreferenced using a stored tile cache on each game object. Closes #1289

cybersphinx commented

The visual impression is kind of hectic, unsettled, when units move. Smoother transitions might be nice, maybe with the radius increased by half a tile or something to offset the delayed "uncovering".

Per commented

I was thinking of perhaps using the same system as when exploring tiles. Then darkness is slowly lifted instead of suddenly disappearing. However, it requires some reworking and re-purposing of that code, which is rather ugly.

Zarel changed status from closed to reopened

Zarel changed resolution from fixed to ``

Zarel commented

Backport to 2.3?

Zarel commented

I rather like the idea of hectic, unsettled, when units move. It's very useful to get instant updates of vision information.

cybersphinx uploaded file 0001-Backport-visibility-changes-r8900-to-2.3.patch (11.8 KiB)

cybersphinx commented

Replying to Warzone2100/old-trac-import#1289 (comment:7):

I rather like the idea of hectic, unsettled, when units move. It's very useful to get instant updates of vision information.

Have you tried current trunk with and without the patch in #1296?

Zarel commented

Yeah. I prefer it without. StarCraft does it the same way (instant vision update). I guess I'm used to it...

Terminator commented

Is it possibe to make it togglable ? On the same button for exemple 'z'.

Per commented

An argument in favour of smooth transitions is consistency, since it is used for exploration already.

Per changed status from reopened to closed

Per changed resolution from `` to fixed

Ai_Tak commented

Fog of war only shows in the radar minimap if it is set to terrain, not hight or terrain+hight.