Some blending doesn't look good about gemrb HOT 13 OPEN

I still question what we are doing with the palettes. Can you add the BAMs in your screenshots to the issue so it's easier to investigate?

The reason my first suspicion is palette related, is because prior to ea4c25f we used to get a similar bad look for projectiles. See 21c16d0 for additional context.

from gemrb.

If it's not a palette issue, then my next guess would be using a different blend mode as you suggest. Projectiles use a two bits to determine which mode to use so we may need to follow that same logic:

the corresponding blend modes from the comments should be one we have defined here:

I'm not sure that I actually implemented them all on the backend tho.

from gemrb.

It's (IWD2) ACIDH.BAM and CONJUCG.BAM as shown in NI:

from gemrb.

I doubt a bit these two are from projectiles — one is clearly a casting glow (so VVC/BAM in fx_casting_glow).

from gemrb.

I know they aren't projectiles, I'm speculating that they ought to behave in a similar fashion in regards to blending. I'm also pointing out that we used to do things incorrectly with projectiles with their palettes/blending and perhaps we were doing the same type of thing here.

In general, I'm skeptical of the type of palette manipulation I removed in 21c16d0 and I doubt it was the only place we were doing it.

It's (IWD2) ACIDH.BAM and CONURCG.BAM as shown in NI:

since those have a white instead of black background, ONE_MINUS_DST probably isn't the correct mode, but maybe one of the others.

from gemrb.

They're transparent (green), I guess the white is from screenshoting (and the second one is actually CONJUCG.BAM).
For the splash translucency might be enough, but not sure about the casting glow, the brightening could be a visual trick, since the bg is darker.

from gemrb.

No, these need special blending. It's the only way to achieve a gradient blend in the original engine and it's clear from the screenshot that there is no gradient. It should be easy to test. If it's a VVC, just force IE_VVC_BLENDED to see if that does it. Then if not, go tweak the blending used for the IE_VVC_BLENDED case to try the others. It's also possible we need a new blend mode, but I doubt it.

we do have this hack that might need to be tweaked/resolved, but maybe that hack is actually what makes this work with BAMs with a regular color key such as these:

from gemrb.

Maybe it's not relevant, since IWD2 doesn't seem to use VVCs, but I do see that the VVC format has an "alpha" field that we ignore. Not sure if it just isn't used by the originals or what. It's not set on any of BG2s VVCs. Maybe on some of the hardcoded ones tho? 🤷🏻‍♂️

from gemrb.

That resref is unset in all games I have, so I'd sooner rule the mask as unused. Why reserve space for one if you don't actually provide any files for it? And why would you need it if it was constructed at runtime.

from gemrb.

If it were used by anything hardcoded there would still likely be a BAM with the alpha information. Finding it might be a pain since it would be invisible in NI, but I would have suspected a similar name to the BAM its an alpha for.

I don't disagree that it's likely unused, features get axed or have alternatives developed without cleaning up the garbage all the time. But it also wasn't uncommon in that era to embed data files within binaries for speed. Don't we already deal with instances of that?

For now it doesn't matter. The obvious first thing to try is the blending.

I see that AA_BLEND does set IE_VVC_BLENDED, so then id try changing the blending used. IIRC, ONE_MINUS_DST was picked because that was what was being used elsewhere. Look at 1ab8bea for context. What is the shadowalpha for? Could that be a problem?

from gemrb.

Let's also figure out which RGBModifiers are being applied to the palette.

from gemrb.

It's for actual shadows, a special case like the transparency index before it. Off the top of my head it's important for avatar animations and inventory items.

from gemrb.

SRC and DST look identical and get very close for casting glows:

Native from first post:

from gemrb.