Closed supsm closed 2 years ago
Wow that's pretty neat. Have you tested it against all the current usage of warm_glow? (I can do it if you haven't)
I've tested a few, but not all. Seems to be fine from the limited testing
My hunch tells me it will break on redstone lamp or glow berries (don't remember if glow berries use it) since they have more brownish color. Of course we can have some compromise, or alternatively make 2 warm glow materials for different tolerance.
I think you're on the right track by testing it on FmN tho. Lumi or Canvas Standard didn't make the difference very obvious.
Yeah, I'd been playing with it recently and noticed my campfire looked a little off. Really highlights the emissivity well
You're right, but the glow berries actually have the opposite effect: the brown parts between the berries are emissive. Redstone lamp has a few spots that look weird. Looks like I need to do some more tweaking
the brown parts between the berries are emissive
You had one job... /j
Redstone lamp is probably beyond saving. It's fine if you don't support it. It probably needs a "warm_dark_glow"
Actually, it'd probably be fine to make the entire redstone lamp emissive when it's on using sprite predicate. I'll do that after the merge.
Redstone lamp still has some issues, glow berries are fixed
Also a random sidenote: redstone torch looks pretty awful (it uses the redstone material)
Why is the redstone lamp material applied when the lamp is off :concern:
Nice. Here are things that uses warm_glow btw
campfire ✔ blast_furnace cave_vines (glow berries) ✔ cave_vines_plant (bottom glow berry) ✔ furnace lantern magma_block redstone_lamp (unnecessary, will change) torch ✔ wall_torch ✔ blaze (could simply be emissive rather than warm_glow?) glow_item_frame (?)
blast furnace, furnace, lantern, and magma block look good. not sure about lantern... it looks a little weird but at the same time I don't really see how it could be better. glow item frame does not work.
blaze also looks fine
Neat. Is this ready to merge then?
Probably. after merging, redstone material (redstone.frag) and redstone lamp should be changed
warm_glow currently has some issues with allowing things like wood to be emissive, which looks really weird. The detection algorithm is changed from subtraction to comparing the ratios Before:
After:
![2022-05-18_17 13 29 jpg](https://user-images.githubusercontent.com/60897356/169176573-bc18fe2f-277f-4890-98c1-31fe96eaec5c.jpg)
Don't think it has any unintended side effects, but jack-o-lanterns still look a bit weird since they're fully emissive (but it's always been this way)