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

Add pixel-size/image-size control [in progress] #7

Open
sk3lk0 opened this issue Oct 29, 2019 · 4 comments
Open

Add pixel-size/image-size control [in progress] #7

sk3lk0 opened this issue Oct 29, 2019 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@sk3lk0
Copy link

sk3lk0 commented Oct 29, 2019

can you add function for change number of pixels?
can you add function for change image size?

@sighook sighook added the enhancement New feature or request label Nov 1, 2019
@sighook
Copy link
Owner

sighook commented Nov 1, 2019

can you add function for change number of pixels?

Seems like there is no problem to make that on some image formats.
Here is the code example: https://github.com/chinarulezzz/pixload/blob/master/gif.pl#L64

It's a "good first issue" for the jpg.pl and png.pl improvement.

The problem appears when we want to specify the pixel size for the "polyglot" images (bmp/gif), where comment symbols /* overwrite pixel information field. A know how to reduce pixel-size, but your proposition needs some research.

can you add function for change image size?

That's non-trivial because of "polyglot" images mentioned above, and payload length that affects image size.

I think that with the addition of pixel-size/payload control we can indirectly control image size. So far in the case of jpg/png. Polyglot images (as i mentioned above) needs some research.

@sk3lk0
Copy link
Author

sk3lk0 commented Nov 1, 2019

thanks.

@sk3lk0 sk3lk0 closed this as completed Nov 1, 2019
@sighook sighook reopened this Nov 1, 2019
@sighook
Copy link
Owner

sighook commented Nov 1, 2019

@sk3lk0 reopened the issue 'cause I can improve when will have time.

@sighook sighook changed the title 2 questions pixel-size/image-size control Nov 1, 2019
@sighook
Copy link
Owner

sighook commented Dec 6, 2021

The pixel-size control currently is implemented in

  • pixload-png
  • pixload-gif
  • pixload-jpg
    Skipped since the payload is injected into DQT table or as a comment. Just inject into existing jpg image with desired pixel width/height.
  • pixload-bmp
  • pixload-webp

@sighook sighook pinned this issue Dec 6, 2021
@sighook sighook mentioned this issue Dec 25, 2021
@sighook sighook changed the title pixel-size/image-size control Add pixel-size/image-size control [in progress] Dec 25, 2021
@sighook sighook self-assigned this Dec 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants