id	summary	reporter	owner	description	type	status	priority	milestone	component	resolution	keywords	cc	phab_field
6546	Shroud of Darkness (SoD) visible terrain and territory lines	Langbart		"An issue was reported via IRC 0ad-dev [https://irclogs.wildfiregames.com/%230ad-dev/2022-05-28-QuakeNet-%230ad-dev.log 28/May/22]

{{{
07:09 < Freagarach_afk> No, that I see the terrain and territory lines in SoD.
07:11 < Freagarach_afk> It seems it depends on the map also.
07:11 < Freagarach_afk> With some terrains it is more visible.
}}}

GL ARB Smoothlos=true Add to surexpose to make it more visible.
Before reveal map
[[Image(ticket:6546:screenshot0018.png, 600px, center)]]
After reveal map
[[Image(ticket:6546:screenshot0022.png, 600px, center)]]

==== to reproduce

- Switch to GL ARB backend
- Restart the Game (Make Sure Smoothlos is on)
- Open Acropolis bay,
- move around and notice everything is black. 
- Open the dev overlay
- Reveal Map, and hide it again.
- move around and notice everything is still slightly visible.


==== bisect 
Was likely triggered after the LOS changes.
[26906]

~~Note: macOS - [26908] I could not reproduce the problem and only created the ticket so that it would not be forgotten.~~ see comment:5:ticket:6546

==== possible solution
Revert the default setting for `smoothlos` back to `false`.


==== reproducible
Adding `smoothlos = ""true""` to the `user.cfg` file and testing on older versions. The problem could be reproduced up to `Alpha XVI - Patañjali` [15148]. Below that I have not tested.


==== additional nit-pick
`smoothlos` was unintentionally removed from the options with [25581]. \\
See IRC 0ad-dev [https://irclogs.wildfiregames.com/%230ad-dev/2022-06-02-QuakeNet-%230ad-dev.log 2/Jun/22]/[https://irclogs.wildfiregames.com/%230ad-dev/2022-06-03-QuakeNet-%230ad-dev.log 3/Jun/22] 

{{{
17:16 < Langbart> @tell wraitii was intended to remove smoothlos with changeset rP25581?
[...]
 07:16 < wraitii> @tell langbart: re rP25581, it looks like it wasn't, but I think vlad recently activated by default (asked him), so maybe the bug can now be seen as  'invalid'
}}}

It may or may not be added back into the options, that is at the judgement of the patch maker."	defect	new	Should Have	Alpha 26	Core engine