[Bug 24857] New thumbnails are not sharp

2010-09-29 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=24857

Ryan Kaldari rkald...@wikimedia.org changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution||FIXED

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 24857] New thumbnails are not sharp

2010-08-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=24857

--- Comment #7 from Tim Starling tstarl...@wikimedia.org 2010-08-20 08:06:05 
UTC ---
The issue is an arbitrary change in SharpenImageChannel() made some time
between ImageMagick 6.4.9 and 6.5.1. The relevant code changed from this:

  alpha=exp(-((double) u*u+v*v)/(2.0*MagickSigma*MagickSigma));
  kernel[i]=(double) (-alpha/(2.0*MagickPI*MagickSigma*MagickSigma));
  if ((width  3) || (u != 0) || (v != 0))
normalize+=kernel[i];

To this:

  alpha=exp(-((double) u*u+v*v)/(2.0*MagickSigma*MagickSigma));
  kernel[i]=(double) (-alpha/(2.0*MagickPI*MagickSigma*MagickSigma));
  normalize+=kernel[i];

The result is that for a given sigma, images look more blurry in the new
convert. Based on brief testing, to reproduce a result similar to the old 0x0.4
requires a sharpen parameter of approximately 0x0.8 in the new ImageMagick. 

Both -unsharp and -sharpen exist in both versions, and have completely
different implementations. Unsharp does not appear to have changed
significantly, so it might be a candidate for version-independent operation in
the next MediaWiki release. But for now I will just change our configuration to
use 0x0.8 instead of 0x0.4.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 24857] New thumbnails are not sharp

2010-08-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=24857

--- Comment #8 from Ryan Kaldari rkald...@wikimedia.org 2010-08-21 00:47:34 
UTC ---
Nice sleuthing!

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 24857] New thumbnails are not sharp

2010-08-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=24857

Derk-Jan Hartman hart...@videolan.org changed:

   What|Removed |Added

 CC||hart...@videolan.org

--- Comment #1 from Derk-Jan Hartman hart...@videolan.org 2010-08-19 17:11:12 
CEST ---
Perhaps syntax change in imagemagick commandline ?

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 24857] New thumbnails are not sharp

2010-08-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=24857

--- Comment #2 from Ryan Kaldari rkald...@wikimedia.org 2010-08-19 17:57:33 
UTC ---
The correct current syntax is using the -unsharp option after the -thumbnail
option. I think previously we were using a radius of 0.8 or something like
that, so:
convert original.jpg -thumbnail 250x90 -unsharp 0.8 thumbnail.jpg

I have no idea if this is different than older syntaxes or what syntax we're
using currently.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 24857] New thumbnails are not sharp

2010-08-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=24857

--- Comment #3 from Ryan Kaldari rkald...@wikimedia.org 2010-08-19 18:25:05 
UTC ---
Ah, looks like the old syntax was -sharpen (which is a much more logical name).
Apparently they're using an unsharp masking algorithm to do the sharpening now,
so they decided they had to change the option name while they were at it!

I've fixed it in r71304.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 24857] New thumbnails are not sharp

2010-08-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=24857

Ryan Kaldari rkald...@wikimedia.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED
 AssignedTo|wikibug...@lists.wikimedia. |rkald...@wikimedia.org
   |org |

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 24857] New thumbnails are not sharp

2010-08-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=24857

Ryan Kaldari rkald...@wikimedia.org changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |

--- Comment #4 from Ryan Kaldari rkald...@wikimedia.org 2010-08-19 18:43:22 
UTC ---
Nevermind, we should be able to use -sharpen or -unsharp and they both should
accept the parameter we're using (0x0.4). Hmm, back to the drawing board.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 24857] New thumbnails are not sharp

2010-08-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=24857

Ryan Kaldari rkald...@wikimedia.org changed:

   What|Removed |Added

 AssignedTo|rkald...@wikimedia.org  |tstarl...@wikimedia.org

--- Comment #5 from Ryan Kaldari rkald...@wikimedia.org 2010-08-19 18:45:42 
UTC ---
reverted self in r71309.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 24857] New thumbnails are not sharp

2010-08-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=24857

--- Comment #6 from Ryan Kaldari rkald...@wikimedia.org 2010-08-19 19:08:28 
UTC ---
Maybe someone with command line access to our ImageMagick could do some tests
and find out what's going on with it, i.e. test using 'convert -thumbnail' with
-sharpen on and off, and then test it with -unsharp on and off, and see if
there are any differences in the output files.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l