Last modified: 2012-10-17 17:22:31 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T26228, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 24228 - Memory problem with large progressive jpegs (on Commons and other wikis)
Memory problem with large progressive jpegs (on Commons and other wikis)
Status: RESOLVED DUPLICATE of bug 17645
Product: Wikimedia
Classification: Unclassified
General/Unknown (Other open bugs)
unspecified
All All
: Low normal with 2 votes (vote)
: ---
Assigned To: Nobody - You can work on this!
http://commons.wikimedia.org/wiki/Com...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-07-02 09:59 UTC by User:Docu
Modified: 2012-10-17 17:22 UTC (History)
7 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description User:Docu 2010-07-02 09:59:47 UTC
With the current configuration, jpegs saved as progressive jpegs ([[JPEG#JPEG compression]]) sometimes don't display if the image is large. 

As compression is probably more useful for large images and if we want to support that, it might be worth finding a solution for larger images.
Comment 1 Ilmari Karonen 2010-07-02 13:25:20 UTC
This would require either patching ImageMagick to use less memory when scaling such images (which may ultimately require patching libjpeg) or switching to a different image scaler.

In principle, there's no reason why downscaling an image in any format couldn't be done using space (at most) proportional only to the size of the target image, not of the source.  However, this requires being able to read and decode the source image data without buffering it all in memory.  For progressive JPEGs this would mean scaling each pass as it is read, and only combining them after scaling.

I'm not aware of any existing image scaler that would do this, but then, I haven't really looked, either.  If anyone knows of a program that does this and otherwise fits our needs (i.e. open source, usable from the command line, actively maintained, etc.), please tell us so we can evaluate it.

It would also be even more useful to have such a scaler for PNG images.
Comment 2 Bawolff (Brian Wolff) 2010-07-02 16:17:15 UTC
>It would also be even more useful to have such a scaler for PNG images.

btw, thats bug 9497.
Comment 3 User:Docu 2010-07-02 17:40:33 UTC
As the image quality for standard thumbnails needn't be perfect, maybe these could be created by decoding only part of the file.
Comment 4 User:Docu 2010-09-06 17:06:20 UTC
Maybe the fix for Bug 24978 solved this too. At least the sample image above works.
Comment 5 Marco 2012-08-10 07:33:34 UTC
I created a cat. to keep track of those images. https://commons.wikimedia.org/wiki/Category:Progressive_mode_JPGs_to_be_saved_in_Baseline_mode
Comment 6 Bawolff (Brian Wolff) 2012-08-10 12:12:53 UTC
We should perhaps keep track of if a jpeg is progressive, and not give progressive jpeg's the exemption to the "max image size to scale limit" that they currently enjoy.
Comment 7 Derk-Jan Hartman 2012-08-23 17:05:42 UTC
*** Bug 37367 has been marked as a duplicate of this bug. ***
Comment 9 esby 2012-09-05 07:54:28 UTC
Fixed the case by uploading a new version with non progressive jpeg setting.

Use exiftool to determine if the jpeg is progressive, 
exiftool myimage.jpg | grep Encoding 
should display
Encoding Process                : Progressive DCT, Huffman coding  <-- progressive, not working on commons for now.
Encoding Process                : Baseline DCT, Huffman coding     <-- non progressive ; working on commons.

@Yannf: Do not use this thread for reporting cases, just put them in the category and pm me over irc if you feel they need to be reencoded.
Comment 10 Nemo 2012-10-17 17:22:31 UTC

*** This bug has been marked as a duplicate of bug 17645 ***

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links