Skip to content

feat: WIP Kitty Graphics - #1148

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

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Conversation

Atan-D-RP4
Copy link

@Atan-D-RP4 Atan-D-RP4 commented Jun 16, 2025

Tracking issue #940

The first stage of building up support for kitty's image protocol.

This updates the parser to extract parameters from the APC's and dispatch to an appropriate handler. Basically the pipeline.

Supporting the full protocol requires maintaining a bunch of state and having access to the some the renderer's state. I don't know how to work with the sugarloaf renderer, so direction on that would be appreciated.

Review of the existing changes is also necessary. The current way I'm doing the passthrough is ugly.

- Set up pipeline to dispatch APC sequences
- (regression) - broke apc_c0_st_terminated and apc_empty tests
@raphamorim
Copy link
Owner

Very cool! I think we can maybe try to get in a minimal functional state. Then we can improve in this PR. Would you mind if add commits to this PR? @Atan-D-RP4

@raphamorim
Copy link
Owner

Also, are you on rio discord? so i can ping you there as well any case

@Atan-D-RP4
Copy link
Author

Atan-D-RP4 commented Jun 17, 2025

Very cool! I think we can maybe try to get in a minimal functional state. Then we can improve in this PR. Would you mind if add commits to this PR? @Atan-D-RP4

Please do. It's exactly why I made it a draft PR. The code changes here are terrible should not be in a release or even dev branch.

Also, are you on rio discord? so i can ping you there as well any case

I am. I'm the same guy who prompted the merge for handling of APC's from you back in the middle of... April I think. My '@' is CanopyRaven.

@raphamorim
Copy link
Owner

Please do. It's exactly why I made it a draft PR. The code changes here are terrible should not be in a release or even dev branch.

🙏 could you give me access to your fork, so i can push straight to it

@raphamorim
Copy link
Owner

I am. I'm the same guy who prompted the merge for handling of APC's from you back in the middle of... April I think. My '@' is CanopyRaven.

Nice will ping you there later

@Atan-D-RP4
Copy link
Author

Please do. It's exactly why I made it a draft PR. The code changes here are terrible should not be in a release or even dev branch.

🙏 could you give me access to your fork, so i can push straight to it

You should have it now. Added you as collaborator.

@raphamorim
Copy link
Owner

Nice thank you!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants