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

[Suggestion] New way of representing Palette in human-readable format #256

Open
MasterInQuestion opened this issue Apr 18, 2024 · 4 comments

Comments

@MasterInQuestion
Copy link

MasterInQuestion commented Apr 18, 2024

    E.g. "#F00 #010203 ..."

    Related:
    https://trac.ffmpeg.org/ticket/10919
    https://trac.ffmpeg.org/ticket/10918

    Does similar command make sense..?
    exiftool -U -ee3 -g3:5:2 -api "RequestAll=0" -api "ByteUnit=Binary" -api "LargeFileSupport" "https://upload.wikimedia.org/wikipedia/en/f/ff/NES_palette_sample_image.png"
    ^
    curl -L "https://upload.wikimedia.org/wikipedia/en/f/ff/NES_palette_sample_image.png" | exiftool -U -ee3 -g3:5:2 -api "RequestAll=0" -api "ByteUnit=Binary" -api "LargeFileSupport" -
    Probably needs better display for FileSize.

@StarGeekSpaceNerd
Copy link
Collaborator

Maybe this thread on the exiftool forums, started by user blue-j, will extract PNG palette in a form you can use?

@MasterInQuestion
Copy link
Author

    Thanks for linking.
    Though my posts are mostly for theoretical enhancement, than looking for workaround.

    Clarification:
    The suggestion is more focused on printing.

@boardhead
Copy link
Contributor

This is an interesting theoretical suggestion, but I would rather not make changes like this unless there is a real demand because it may break things for users already relying on the way this tag is decoded.

But I'll patch ExifTool 12.87 to avoid reporting FileSize for a pipe.

  • Phil

@MasterInQuestion
Copy link
Author

    "(Binary data 768 bytes, use -b option to extract)"
<^>    The interpreted display is regardless susceptible to change.
    And this one would be mostly just verbatim copied.

    Perhaps displaying the parsed file size makes more sense.

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

No branches or pull requests

3 participants