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

Libsixel forked #1756

Closed
dnkl opened this issue Jun 12, 2021 · 3 comments
Closed

Libsixel forked #1756

dnkl opened this issue Jun 12, 2021 · 3 comments
Assignees
Labels
userquestion not quite bugs--inquiries from users
Milestone

Comments

@dnkl
Copy link
Contributor

dnkl commented Jun 12, 2021

This is just an FYI: Arch's libsixel AUR package just switched to a libsixel fork. Mentioning this since I know you've opened issues in the original libsixel project.

More info: saitoha/libsixel#154

@dnkl dnkl added the userquestion not quite bugs--inquiries from users label Jun 12, 2021
@dnkl
Copy link
Contributor Author

dnkl commented Jun 12, 2021

... and just saw you're co-maintining it :)

@dnkl dnkl closed this as completed Jun 12, 2021
@dankamongmen
Copy link
Owner

dankamongmen commented Jun 12, 2021

yep =] i'm also packaging it for Fedora, or at least am halfway through doing so. i aborted the effort when i decided not to use it, but will likely pick it back up now. just made a bug on this: libsixel/libsixel#11

i'd like to use it, for general code-reuse and capability-pooling reasons.

related bugs: #639, #641, and #1391

as i recall, libsixel primarily wants to work by you pumping in pixels, and it giving you back a sixel-encoded string. that's kinda suboptimal for us; i'd have to write a sixel decoder. not a difficult thing by any means, but that functionality probably belongs in libsixel. what i'd ideally get back (in addition to the string) is a matrix of palette indices (size defined by number of color registers, or some max size) and a matrix of equal dimensions (single bit values) for transparency. obviously this is trivially built if one decodes the generates sixel string, but it's probably faster (at least for large images) to generate it while generating said string, due to better cache behavior in a single-pass approach.

tagging @ctrlcctrlv so he has visibility.

@dankamongmen
Copy link
Owner

i'd also, of course, need it to be on all our supported platforms.

@dankamongmen dankamongmen added this to the 3.0.0 milestone Jun 12, 2021
@dankamongmen dankamongmen self-assigned this Jun 12, 2021
@dankamongmen dankamongmen modified the milestones: 3.0.0, 2.4.0 Aug 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
userquestion not quite bugs--inquiries from users
Projects
None yet
Development

No branches or pull requests

2 participants