Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

investigate strange behavior in stacking tests #1318

Closed
dankamongmen opened this issue Feb 1, 2021 · 11 comments
Closed

investigate strange behavior in stacking tests #1318

dankamongmen opened this issue Feb 1, 2021 · 11 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@dankamongmen
Copy link
Owner

See #1068 and #1317 for background info. I've added some Stacking unit tests which verify transparent blitter stacking on quadblitters. It appears that the first plane alone is generating a full block, as opposed to the crossed quadrant we would expect. As a result, our unit test is invalid. Add a new ncvisual_from_rgba() unit test checking this case, and it ought break. Fix the breakage.

@dankamongmen dankamongmen added the bug Something isn't working label Feb 1, 2021
@dankamongmen dankamongmen added this to the 2.2.0 milestone Feb 1, 2021
@dankamongmen dankamongmen self-assigned this Feb 1, 2021
@dankamongmen
Copy link
Owner Author

hrmmm, we have plenty of NCBLIT_2x2 ncvisual_from_rgba() unit tests. I think the problem in the Stacker tests is something else.

@dankamongmen
Copy link
Owner Author

htole(0xffffffff), htole(0xff000000), htole(0xff000000), htole(0xffffffff) results in the expected cross, but replacing htole(0xff000000) with htole(0) gets us a full block...i don't understand that?

@dankamongmen
Copy link
Owner Author

are we sure qtrans_check() is right for the ▚ glyph?

@dankamongmen
Copy link
Owner Author

are we sure qtrans_check() is right for the ▚ glyph?

nope, this looks right, and is checked by unit test

@dankamongmen
Copy link
Owner Author

with that said, i'm pretty sure ncvisual_from_plane() does not work with NCBLIT_2x2.

@dankamongmen
Copy link
Owner Author

is the ncplane* n_ coming in with a full block?!?

@dankamongmen
Copy link
Owner Author

Sure enough, we're getting a full block in the n_ plane somehow, and ncplane_erase() doesn't eliminate it...does it possibly have a base cell? I don't understand what's going on here.

@dankamongmen
Copy link
Owner Author

yeah, it definitely seems to be coming in from the previous test somehow. changes made there, colorwise, are reflected....

@dankamongmen
Copy link
Owner Author

ncplane_at_yx() verifies ▚ at top[0, 0]. is this possibly related to #1314? is there possibly an error early in the rasterizer?

@dankamongmen
Copy link
Owner Author

our channel from the ncplane_at_yx_cell() is c1ffffff60000000

@dankamongmen dankamongmen changed the title ncvisual_from_rgba() works improperly with NCBLIT_2x2 investigate strange behavior in stacking tests Feb 2, 2021
@dankamongmen
Copy link
Owner Author

OK, i circled back around to this, and the StackedQuadCrossed unit test, when modified as outlined earlier, no longer generates an early full block, but instead the expected cross. I'm guessing this got fixed by some of the other stacking work. Closing as fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant