Patch Set 91:
What's the maximum file size?

There's no strict limit, just keep in mind that these files will remain in coreboot's git history forever (long after the board is dead, the code removed and the documentation pruned), so as small as is reasonable for the content?

Try different compression values and see what configuration is the smallest that still provides a visually pleasing representation (ie. not blocky pixel soup).

I played around a bit: top.jpg can be compressed better (fewer sharp edges because there's no text), and at q50 I get a reasonable looking image at 29K. bottom.jpg requires a bit more and looks good to me at q70, which brings it to 45K. At q50 (33K) the "pass" sticker looks slightly odd - but if you think that's good enough, by all means go down to that level :-)

View Change

To view, visit change 31378. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ia913534ec176fc600fcd4ce3af335ebe682b0ed4
Gerrit-Change-Number: 31378
Gerrit-PatchSet: 91
Gerrit-Owner: Felix Singer <felixsinger@posteo.net>
Gerrit-Reviewer: Felix Singer <felixsinger@posteo.net>
Gerrit-Reviewer: Krystian Hebel <krystian.hebel@3mdeb.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Michał Żygowski <michal.zygowski@3mdeb.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph@9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki@gmail.com>
Gerrit-Reviewer: Piotr Król <piotr.krol@3mdeb.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Reviewer: ron minnich <rminnich@gmail.com>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Mon, 20 May 2019 15:29:30 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment