View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0003067 | The Dark Mod | Coding | public | 22.03.2012 14:39 | 05.05.2012 18:07 |
Reporter | Springheel | Assigned To | tels | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Product Version | TDM 1.07 | ||||
Target Version | TDM 1.08 | Fixed in Version | TDM 1.08 | ||
Summary | 0003067: Shadow comes on before light disappears | ||||
Description | When torches and candles go out, something weird happens. The torches suddenly cast a _huge_ shadow for a second. It's as if the flame you just extinguished is casting a shadow even though the particle is gone. It then blinks out The candles also do something weird when extinguished. For a second there is a sliver of light on the wall which then disappears. Might be the same cause as the torch shadow. http://forums.thedarkmod.com/topic/13564-lamp-glass/page__pid__281809__st__25#entry281809 | ||||
Additional Information | As for the shadow "pop", I think that can be mitigated by the code delaying turning on the shadow until the light has gone out | ||||
Tags | No tags attached. | ||||
Just so this doesn't get forgotten: The "delay the shadow until the light is off" is actually still a work-around in the form of "have a shadow only when unlit". A much better solution would be if we could tag certain entities with "no self shadowing from attached lights" and then a light will simply not cast a shadow from any entities on the same bind team. Right now, entities either cast a shadow from a light source (if the light source is shadow casting and the entity, too), or not, but they can't selectively have a shadow from light X, and not from light Y. Implementing this in the engine might be complicated, or dead simple, but I don't have enough understandings of the innards to deduce if that would be possible. But having that ability would be a much cleaner solution, because then we could remove the entire shadow switching thing altogether. |
|
The light holder script was fixed with revision #12772 to toggle the shadow on with a delay of 100ms, but I cannot test this (TDM no longer runs on linux for me). Can somebody please report back? | |
Tried some candles in light_entity.map. The candles without holders still cast shadows before the light went off. So did the candle in a bottle. But the candles in the typical holders seemed to be fixed. |
|
Could you make a very small testmap with just the three entities where one can observe the difference? Technically, all use the same light holder script, so all should show the same behaviour. |
|
On more testing, it seems like the shadows on the candle holders are fixed. But the candles themselves still cast shadows if they're close enough to a wall to do so. | |
It also appears that torches are still exhibiting the problem behaviour. | |
I was wrong, the delayed shadow was only used in the script for the ligh entities itself, but not in the script for the light holders. Oops :) This is now fixed, and should now work. If not, please re-open this bug report. |
|
I tested torches and campfires, and the issue is still there. It might be for a slightly shorter period of time than before (hard to say). |
|
grayman says: "I tried 200 and could still see the shadow. Then I tried 400 and couldn't see it. I tested a few different types of light holders and saw no issues dousing or lighting them. Checked in, rev 12798." |
|
One issue is that now electric light shadows don't come on for half a second. Since electric lights turn off instantly, the delay in the shadow coming on is noticaeble. Given the choice, though, I'd rather have this problem than the other. | |
Date Modified | Username | Field | Change |
---|---|---|---|
22.03.2012 14:39 | Springheel | New Issue | |
22.03.2012 14:39 | Springheel | Status | new => assigned |
22.03.2012 14:39 | Springheel | Assigned To | => tels |
22.03.2012 18:18 | tels | Severity | normal => minor |
22.03.2012 18:18 | tels | Product Version | TDM 1.08 => TDM 1.07 |
22.03.2012 18:18 | tels | Target Version | => TDM 1.08 |
22.03.2012 18:24 | tels | Note Added: 0004424 | |
16.04.2012 17:56 | tels | Assigned To | tels => |
16.04.2012 17:56 | tels | Status | assigned => new |
20.04.2012 09:20 | tels | Note Added: 0004488 | |
20.04.2012 09:20 | tels | Assigned To | => tels |
20.04.2012 09:20 | tels | Status | new => feedback |
21.04.2012 00:37 | Springheel | Note Added: 0004490 | |
21.04.2012 00:37 | Springheel | Status | feedback => assigned |
21.04.2012 00:38 | Springheel | Note Edited: 0004490 | |
22.04.2012 08:45 | tels | Note Added: 0004491 | |
22.04.2012 12:51 | Springheel | Note Added: 0004492 | |
27.04.2012 01:24 | Springheel | Note Added: 0004497 | |
01.05.2012 14:04 | tels | Note Added: 0004505 | |
01.05.2012 14:04 | tels | Status | assigned => resolved |
01.05.2012 14:04 | tels | Fixed in Version | => TDM 1.08 |
01.05.2012 14:04 | tels | Resolution | open => fixed |
02.05.2012 01:16 | Springheel | Note Added: 0004513 | |
02.05.2012 01:16 | Springheel | Status | resolved => assigned |
02.05.2012 01:17 | Springheel | Note Edited: 0004513 | |
02.05.2012 01:22 | Springheel | Note Edited: 0004513 | |
02.05.2012 18:52 | tels | Note Added: 0004517 | |
02.05.2012 18:52 | tels | Status | assigned => resolved |
05.05.2012 18:07 | Springheel | Note Added: 0004545 |