Fix incorrect pixel colors (in tinygl.obj) #270

Open
opened 2025-07-06 18:17:55 +00:00 by IgorA · 2 comments
Member

Need to fix incorrect pixel colors. Happens due to overflow when calculations result in values ​​greater than 255.

Need to fix incorrect pixel colors. Happens due to overflow when calculations result in values ​​greater than 255.
31 KiB
IgorA added the Category/LibrariesFASM labels 2025-07-06 18:17:56 +00:00
Contributor

Hi @IgorA,

I built and ran test_glu1 from TinyGL/asm_fork/examples on QEMU but I'm unable to reproduce the incorrect pixel colors. The rendering looks correct to me.

Could you please clarify:

Has this issue already been fixed?
Are there specific steps to reproduce the issue (e.g. rotating to a certain angle, resizing the window)?

Thanks!

image.png

Hi @IgorA, I built and ran `test_glu1` from `TinyGL/asm_fork/examples` on QEMU but I'm unable to reproduce the incorrect pixel colors. The rendering looks correct to me. Could you please clarify: Has this issue already been fixed? Are there specific steps to reproduce the issue (e.g. rotating to a certain angle, resizing the window)? Thanks! ![image.png](/attachments/f5b30864-2963-4d66-aa03-7f5d6f7ef40b)
Author
Member

Hello, hrigar

The problem isn't solved. If you run test_glu1 or another program, you can find certain angles at which the incorrectly colored pixels are visible. For example, open test_glu1, press left (1 time) and up (17 times), and a black spot becomes visible on the red cone. Coordinates z -48, y -99.

Coordinates z -84, y -150 aren't visible because I resized the window in the last update.

Thanks!

Hello, [hrigar](https://git.kolibrios.org/hrigar) The problem isn't solved. If you run test_glu1 or another program, you can find certain angles at which the incorrectly colored pixels are visible. For example, open test_glu1, press left (1 time) and up (17 times), and a black spot becomes visible on the red cone. Coordinates z -48, y -99. Coordinates z -84, y -150 aren't visible because I resized the window in the last update. Thanks!
Sign in to join this conversation.
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: KolibriOS/kolibrios#270