search

pyrollo / display_modpack

Display modpack for Minetest, provides mods with dynamic display and font display : clocks, signs, and more.
GNU Lesser General Public License v3.0
25 stars 27 forks source link

Strange text color/lighting behavior #35

Closed tuedel closed 5 years ago

tuedel commented 5 years ago

I've been experiencing a strange rendering bug since MT 5.0 where the actual text color of a display entity seems to be dependent on the environment light level at the time of its last update, and generally seems to change in weird/unexpected ways.

It seems to affect certain types of signs/displays more than others depending on their text color. I first noticed it with signs_road:blue_street and the scifi screens from https://github.com/pyrollo/digiterms, which also seem to exhibit a slightly different behavior regarding light level changes:

display_lighting_bug (It's the same for artificial light, it was just easier to demonstrate using /time)

I'm not sure if this is a bug in the engine or in display_api. Any ideas? :)

pyrollo commented 5 years ago

Hi tuedel, That's interesing. When moving to MT5 I had the opposite bug : entity kept being lighten until hit. I guess that has been fixed in MT5, inducing this behavior. Something must be missing in the entity spawn code.

tuedel commented 5 years ago

Salut @pyrollo! :)

The demonstration above was recorded using a fairly recent build of MT 5.1.0-dev (https://github.com/minetest/minetest/commit/1db3d252cff9e8d61fecf1052d7497813851da51)

edit: Oops, I misread your comment a bit.

tuedel commented 5 years ago

After some bisecting I found https://github.com/minetest/minetest/commit/787cd15c14b65f50b06c6b9ccecf28e937318430 (https://github.com/minetest/minetest/pull/6559) to be responsible for this.

Before that commit everything looks fine to me, I was unable to reproduce the initial bug you describe.

tuedel commented 5 years ago

Issue: https://github.com/minetest/minetest/issues/9020 Fix: https://github.com/minetest/minetest/pull/9021

pyrollo commented 5 years ago

Thanks for that investigation !

Good to know this will be solved soon :)