View Issue Details

IDProjectCategoryView StatusLast Update
0003842The Dark ModTexturespublic10.09.2014 21:14
ReporterSpringheel Assigned ToSpringheel  
PrioritynormalSeveritynormalReproducibilityhave not tried
Status resolvedResolutionfixed 
Product VersionTDM 2.03 
Fixed in VersionTDM 2.03 
Summary0003842: put alpha textures on separate layer
DescriptionApparently layers that have an alpha stage don't get affected by wound decals. This is why the citywatch tunic and townsfolk_female don't spawn wound decals on their torso.

The alpha texture should be kept to its own layer when possible.

TagsNo tags attached.

Activities

Springheel

Springheel

07.09.2014 12:46

administrator   ~0006954

Just occurred to me that, in the citywatch case, I could just model the ring in the correct shape for the high-poly version. That wouldn't require any changes to layers or materials. On the LOD stages the mistake would be less noticable--in fact, if you paint the transparent area in black, then that would probably be fine.
SteveL

SteveL

07.09.2014 20:05

reporter   ~0006958

Aha! So make a genuine hole in the mesh rather than use alpha testing to fake a hole? That sounds good.
SteveL

SteveL

07.09.2014 20:54

reporter   ~0006959

I was just looking at the code in Material.cpp to see whether we could consider a new keyword for .mtr files, "allowOverlays", to cover cases where a material only has small alpha "holes" and should be ok to receive a blood splat, but it looks like one already exists: "forceOverlays". Not tried it yet, but there's code for it.
Springheel

Springheel

08.09.2014 02:38

administrator   ~0006960

That's definitely worth a test.
SteveL

SteveL

08.09.2014 06:48

reporter   ~0006962

Seems to work. I added the alphatest to citywatch_cloth and my fix stopped working--no more blood. Then I added forceOverlays and it started working again.

You won't see it work on the guard until I've committed the fix, which I'll do tonight. It should work immediately for the townsfolk_female assuming she doesn't need reskinning to remove chainmail.
SteveL

SteveL

08.09.2014 17:53

reporter   ~0006968

Committed the fix + binaries so forceOverlays should now fix both the townsfolk_female and the cloth-wearing guard.
Springheel

Springheel

08.09.2014 21:17

administrator   ~0006970

Cool! I'll make those changes to the material defs if you like.
SteveL

SteveL

09.09.2014 06:49

reporter   ~0006971

Yes please. I didn't commit any defs for fear of doing half a job.

Issue History

Date Modified Username Field Change
06.09.2014 20:39 Springheel New Issue
06.09.2014 20:39 Springheel Status new => assigned
06.09.2014 20:39 Springheel Assigned To => Springheel
07.09.2014 12:46 Springheel Note Added: 0006954
07.09.2014 20:05 SteveL Note Added: 0006958
07.09.2014 20:54 SteveL Note Added: 0006959
08.09.2014 02:38 Springheel Note Added: 0006960
08.09.2014 06:48 SteveL Note Added: 0006962
08.09.2014 17:53 SteveL Note Added: 0006968
08.09.2014 21:17 Springheel Note Added: 0006970
09.09.2014 06:49 SteveL Note Added: 0006971
10.09.2014 21:14 Springheel Status assigned => resolved
10.09.2014 21:14 Springheel Fixed in Version => TDM 2.03
10.09.2014 21:14 Springheel Resolution open => fixed