After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 343576 - Lanczos resize not "the best"
Lanczos resize not "the best"
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: General
2.3.x
Other Linux
: High critical
: 2.4
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2006-06-01 10:22 UTC by techno
Modified: 2008-01-15 14:06 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description techno 2006-06-01 10:22:08 UTC
1. Create a big image (around 3000px x 2000px)
2. Add a text layer, us a small text, preferbly a non-standard font, text layer shoudl be around 70-80px high. (ie. http://www.fontpool.com/samples/linotype/linotype_zapfino/familyid/5970/4/default.gif?width=350 http://www.fontpool.com/samples/gideon/passions_conflict/familyid/50035/4/default.gif?width=170 ).
3. Resize down to 1024x... or similar, using Lanczos.
4. Undo resize, and reapeat using cubic. 
5. Comapre Results, Lanczosc producesz jaggy text, Cubic makes it look normal.
Comment 1 Sven Neumann 2006-06-01 10:46:07 UTC
Actually, there should be no difference at all when scaling down. But the Lanczos implementation is known to be buggy and will most likely be removed unless someone fixes it really soon now.
Comment 2 Sven Neumann 2006-06-27 19:08:58 UTC
We need to look at this before the 2.4 release. Lanczos scaling absolutely needs to work or it will only confuse people.
Comment 3 Sven Neumann 2006-06-27 19:09:50 UTC
Oops, didn't meant to change the status. Changing it back...
Comment 4 techno 2006-06-27 20:39:12 UTC
I can retry the procedure with the newest release, or maybe CVS (if it's accesible), and tell you the results. Maybe I'll be able to compile with EXIF support now. 
Comment 5 weskaggs 2006-08-22 18:08:42 UTC
It seems to me that the summary says it all: Lanczos is not "the best" in all situations.  In effect, Lanczos is something like cubic plus a bit of unsharp-mask.  That is helpful in many situations but not in all, particularly when downscaling.  The first thing that needs to be done, then, is to remove "best" from the menu of interpolation options, which I have done.

For this bug report, I think that is probably enough.  Note that there are other bug reports concerning Lanczos (particularly bug #167956) that remain open and urgently need to be solved for 2.4.

I will leave this open for now, but will resolve as FIXED soon unless somebody objects.

Comment 6 weskaggs 2006-08-22 18:09:18 UTC
2006-08-22  Bill Skaggs  <weskaggs@primate.ucdavis.edu>

	* libgimpbase/gimpbaseenums.c: Don't label Lanczos
	as "best".  Addresses bug #343576.
Comment 7 Sven Neumann 2006-08-23 07:53:49 UTC
2006-08-23  Sven Neumann  <sven@gimp.org>

	* libgimpbase/gimpbaseenums.h: do the change for bug #343576 here;
	the .c file is generated.


Shouldn't we put the label "(Best)" back on the Cubic interpolation method then? Or, alternatively, remove "(Fastest)" from None?
Comment 8 weskaggs 2006-08-23 14:13:24 UTC
Well, I'm not an expert on this stuff, but it looks to me like Lanczos often *is* the best when you're scaling up, at least with photographs.  I would agree with removing (Fastest) from None, though, since in the absence of labels on other things, it could make it seem like None is the "best".  I wonder if we could put tooltips on these things that would allow more useful explanations?

Comment 9 Sven Neumann 2006-08-23 14:18:21 UTC
No, we can't put tooltips on items in combo-boxes. I will remove (Fastest) from "None" then and also add a translation context there.
Comment 10 weskaggs 2006-08-25 17:27:38 UTC
Closing as FIXED.  The remaining issues with Lanczos are better dealt with in bug #167956.

Note that the following change was made to HEAD without comment here:

2006-08-23  Sven Neumann  <sven@gimp.org>

	* libgimpbase/gimpbaseenums.[ch]: removed "(Fastest)" from "None"
	and added translation context (bug #343576).