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

orca is hidden behind deltas in intro on kitty selfref #1990

Closed
dankamongmen opened this issue Jul 26, 2021 · 4 comments
Closed

orca is hidden behind deltas in intro on kitty selfref #1990

dankamongmen opened this issue Jul 26, 2021 · 4 comments
Assignees
Labels
bitmaps bitmapped graphics (sixel, kitty, mmap) bug Something isn't working
Milestone

Comments

@dankamongmen
Copy link
Owner

In intro on Kitty with selfref animation (see #1900), once the orca emerges from behind the HUD, she's missing her midsection. This is a regression, pointing to broken rebuilds. Screenshot:

2021-07-26-182117_1047x1417_scrot

Let's try to get this fixed up before Kitty 0.21.3 drops and triggers this behavior (I'm forcing selfref for the purpose of testing).

@dankamongmen dankamongmen added bug Something isn't working bitmaps bitmapped graphics (sixel, kitty, mmap) labels Jul 26, 2021
@dankamongmen dankamongmen added this to the 3.0.0 milestone Jul 26, 2021
@dankamongmen dankamongmen self-assigned this Jul 26, 2021
@dankamongmen
Copy link
Owner Author

yeah, if one runs bitmapstates, when it says "should see full square", we're still seeing asterisks. rebuilds are definitely broken. that's surprising, because statepixel is working properly.

@dankamongmen
Copy link
Owner Author

hrmmm, i thought it might be due to the image moving, but that doesn't seem to be the problem...i can have statepixel move all over the place, and things remain alright:

2021-07-27-043106_1047x1417_scrot

@dankamongmen
Copy link
Owner Author

ahhh, i think what's going on is that we're emitting the rebuild into the local mstreamfp. we hit sprite_redraw(), and that's going to prefer kitty_move(), which writes directly to the FILE. we need dump the mstreamfp contents into the FILE beforehand, and that ought get it i think.

@dankamongmen
Copy link
Owner Author

this was kovid being sloppy, not our bug =]. verified that things work with his fix during the 0.22.1 development cycle. i'm going to reenable this backend for kitty 0.22.1+.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bitmaps bitmapped graphics (sixel, kitty, mmap) bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant