View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
---|---|---|---|---|---|---|---|---|---|
0005063 | Spring engine | General | public | 2016-02-07 16:09 | 2016-02-07 22:15 | ||||
Reporter | Beherith | ||||||||
Assigned To | Kloot | ||||||||
Priority | normal | Severity | tweak | Reproducibility | always | ||||
Status | closed | Resolution | no change required | ||||||
Product Version | 100.0+git | ||||||||
Target Version | Fixed in Version | ||||||||
Summary | 0005063: ModelFragProg.glsl writes to deferred buffers even if the fragment is fully transparent. | ||||||||
Description | The deferred buffers are written to even if a fragment is transparent. Maybe just an early [ if (extraColor.a < 0.5) discard; ] is sufficient, but I see that there have been some alpha pass changes in the shader code, and am unsure if i'm not breaking some intended functionality with this. | ||||||||
Steps To Reproduce | Requires deferred buffer visualiaztion, and a unit with alpha transparency. For this, I used some trees and BAR's deferred lighting shader. See the attached screenshot for details. | ||||||||
Tags | No tags attached. | ||||||||
Checked infolog.txt for Errors | |||||||||
Attached Files |
|
![]() |
|
Kloot (developer) 2016-02-07 21:09 Last edited: 2016-02-07 21:09 |
extraColor is written to the SPECTEX buffer, so you can kill the lighting for those fragments by multiplying with float(SPECTEX.a > 0.0) on the deferred side (a branch would hurt more than writing the fragment does). |
![]() |
|||
Date Modified | Username | Field | Change |
---|---|---|---|
2016-02-07 16:09 | Beherith | New Issue | |
2016-02-07 16:09 | Beherith | File Added: screen00141.jpg | |
2016-02-07 21:09 | Kloot | Note Added: 0015644 | |
2016-02-07 21:09 | Kloot | Note Edited: 0015644 | View Revisions |
2016-02-07 22:15 | Kloot | Status | new => closed |
2016-02-07 22:15 | Kloot | Assigned To | => Kloot |
2016-02-07 22:15 | Kloot | Resolution | open => no change required |