HeapsIO / heaps

Heaps : Haxe Game Framework
http://heaps.io
MIT License
3.22k stars 340 forks source link

h3d PBR lighting bug in Edge #1008

Open ghost opened 3 years ago

ghost commented 3 years ago

As of 2021-10-21 (Git commit d95a4b1bf000027764769570ce1d4aeb858f7500), the PBR sample does not display correctly in Microsoft Edge: The half-sphere in the direction of +x +z (+red +blue) is all black whenever Light is greater than about 0.4.

The same demo built for Hashlink displays correctly, and the light is visible on that side of the sphere and varies smoothly in intensity throughout the slider's range.

sample page light=0 4211 sample page light=0 3684 sample page light=0

rcstuber commented 3 years ago

I can confirm this also occurs in Chrome and Safari on Mac. No errors being thrown in the console.

ncannasse commented 2 years ago

Ping @Yanrishatum

Yanrishatum commented 2 years ago

@rransom8774 Please confirm that the aforementioned commit exactly breaks the PBR rendering on JS. I highly doubt it is what causes the bug, as it doesn't come even close to touching PBR rendering. Also, please confirm the bug is reproducible on JS builds from samples folder, as I'm not completely confident the samples on heaps.io are properly up-to-date.

ghost commented 2 years ago

Please confirm that the aforementioned commit exactly breaks the PBR rendering on JS. I highly doubt it is what causes the bug, as it doesn't come even close to touching PBR rendering.

That was the latest commit as of when I tested that sample. I did not try to determine which commit caused the problem.

The error reported to me in the Edge console was that an RGBA32F texture could not be created.

Also, please confirm the bug is reproducible on JS builds from samples folder, as I'm not completely confident the samples on heaps.io are properly up-to-date.

I did test the JS builds from that exact commit, and saw exactly the same behavior in that as in the online sample as of that date. I did not intend to take screenshots of the wrong window.