-
Notifications
You must be signed in to change notification settings - Fork 1.9k
image addon serialization #5845
Copy link
Copy link
Open
Labels
area/addon/imagearea/addon/serializearea/performancetype/enhancementFeatures or improvements to existing featuresFeatures or improvements to existing featurestype/planA meta issue that consists of several sub-issuesA meta issue that consists of several sub-issues
Metadata
Metadata
Assignees
Labels
area/addon/imagearea/addon/serializearea/performancetype/enhancementFeatures or improvements to existing featuresFeatures or improvements to existing featurestype/planA meta issue that consists of several sub-issuesA meta issue that consists of several sub-issues
Currently images printed to the terminal cannot be serialized. Previous tests regarding image serialization were done in jerch/xterm-addon-image#48. They revealed the following cornerstones:
Steps to be taken
As a possible cause of actions I suggest the following steps:
The reason for using QOI and not one of the standard formats is mostly performance. QOI yields comparable file sizes to PNG, but is in previous tests ~4 times faster to process.
For the custom sequence we can take bits from the kitty protocol, or extend it for encoding purposes with QOI. The tricky part will be to harmonize it with SIXEL and IIP states though. Currently I don't aim to write a full serialization for all protocol types, as this seems overbloated. Instead we should extract the minimal needed bits to restore images states correctly with one sequence and image format only.
Up for discussion first, as it will introduce quite some changes across the codebase.
FYI: @Tyriar, @anthonykim1, @PerBothner