Optimize PNGs

[scop]

BPO	27461
Nosy	@vstinner, @scop
Files	pngs.patch

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = None
created_at = <Date 2016-07-07.12:29:34.334>
labels = ['type-feature', 'docs']
title = 'Optimize PNGs'
updated_at = <Date 2018-06-16.15:37:41.531>
user = 'https://github.com/scop'

bugs.python.org fields:

activity = <Date 2018-06-16.15:37:41.531>
actor = 'willingc'
assignee = 'docs@python'
closed = False
closed_date = None
closer = None
components = ['Documentation']
creation = <Date 2016-07-07.12:29:34.334>
creator = 'scop'
dependencies = []
files = ['43896']
hgrepos = []
issue_num = 27461
keywords = ['patch']
message_count = 5.0
messages = ['269929', '269930', '269931', '269932', '271377']
nosy_count = 3.0
nosy_names = ['vstinner', 'scop', 'docs@python']
pr_nums = []
priority = 'normal'
resolution = None
stage = 'patch review'
status = 'open'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue27461'
versions = []

Linked PRs

• gh-71648: Remove unused image Doc/faq/python-video-icon.png #138841
• gh-71648: Optimize PNG files #138842
• [3.14] gh-71648: Optimize PNG files (GH-138842) #138851
• [3.13] gh-71648: Optimize PNG files (GH-138842) #138852

[scop]

Running PNGs through zopflipng makes them smaller, with no drawbacks. This patch was done with zopflipng version 1.0.1 with the -m option, and decreases the in-tree total size size of all *.png by almost 100KiB.

$ find -name "*.png" | xargs du -bc | tail -n 1  # before
291101	total

$ find -name "*.png" | xargs du -bc | tail -n 1  # after
194639	total

[vstinner]

Hi, The idea is interesting, but IMO it would be more efficient to contribute to Sphinx, so all documentations will benefit of smallest pictures. There is a similar tool for JPEG. Victor

[scop]

Sure, I plan to suggest this to sphinx as well.

One corresponding tool for JPEGs is jpegoptim and its --strip-all option, but there is only ./Mac/BuildScript/resources/background.jpg in the cpython source tree (which despite of the .jpg extension appears to be a PNG file so I don't want to touch that stuff), plus ./Lib/test/imghdrdata/python.jpg which isn't worth a patch alone :)

[vstinner]

there is only ./Mac/BuildScript/resources/background.jpg in the cpython source tree (which despite of the .jpg extension appears to be a PNG file so I don't want to touch that stuff)

Oh funny, you're right: it's a PNG file :-)

[scop]

Updated patch against tip.

Before: 289426 total
After: 194638 total

[StanFromIreland]

This was already done in #74845

I will check if doing it again has any benefit