Rotating EPS leads to colour loss

[Jan Strube]

I rotated a jpeg image (imported by digikam).
Clicking the right arrow to browse to the next image brought up a pop-up window to ask if I want to save the modified image. I confirmed with <Enter>
Unfortunately this resulted in the image file being of length 0 bytes...

Performing the same actions on an eps image led to its losing its colors...

Luckily I had backups, but I'm a bit disappointed to have a program in my distro that has bugs as serious as this.

[Jan Strube (jan-strube-gmx)]

I already filed a bug upstream

[Phil Bull (philbull)]

Thanks for the report.

Could you link to the upstream bug in this bug report, please?

[Phil Bull (philbull)]

Severity reduced. See https://wiki.ubuntu.com/HelpingWithBugsOld

[Phil Bull (philbull)]

Found it:

http://sourceforge.net/tracker/index.php?func=detail&aid=1414343&group_id=15954&atid=115954

Sorry for the spam.

[Achim Bohnet (allee)]

Hi Jan,
are you sure upstream still watches sf.net bug tracker?
gwenview is developed in KDE svn repo and uses KDE's
bugzilla. See help->report bug in gwenview.

Questions:

o what type of image (jpeg?) was truncated to zero

o In gwenview did you use Edit -> rotate left/right
or Plugins -> image -> rotate?

[Jan Strube (jan-strube-gmx)]

Hi,
as the bug report states I was using jpeg.
I used the rotate button.

Thanks for looking into this.

[Achim Bohnet (allee)]

Hi Jan,

I can't reproduce it currently in up to date dapper.
AFAIR I could last Wednesday (but this was very
late in the night)

Can you try again?

Achim

[Jérémie Corbier (jcorbier)]

As there is no response from the reporter I am closing this bug. Feel free to reopen this bug if the problem still exists in Dapper.

[Jan Strube (jan-strube-gmx)]

Just installed Dapper. Seems to be fixed.
Thanks,
    Jan

[Jan Strube (jan-strube-gmx)]

Aehh. Never mind. Please re-open the bug.
Rotating EPS images as follows leads to color loss.WTF ?
Open EPS image.
click the rotate clockwise button
click the right arrow ('next') button
confirm with enter.
go back and enjoy the corrupted image.

Please tell me you can reproduce this ?
Jan

[Phil Bull (philbull)]

Could you attach a screenshot of the corruption problem, please? And also a small example EPS file if possible.

Also, what version of the package do you have installed?

Thanks

[Jan Strube (jan-strube-gmx)]

This is a screenshot of gwenview after rotating the image.
You can see both versions, because the filefolder preview on the right pane hasn't updated, yet.

[Jan Strube (jan-strube-gmx)]

pic after the faulty transformation.

[Jan Strube (jan-strube-gmx)]

pic before the faulty transformation

[Jan Strube (jan-strube-gmx)]

This is with gwenview 1.3.1-0ubuntu4

[Jan Strube (jan-strube-gmx)]

This is not Ubuntu-specific. I have installed gwenview on another machine with konstruct. The problem exists there also.

[Phil Bull (philbull)]

Thanks Jan, upstream have been very quiet on this issue, so i don't think we're going to get a fix any time soon. I've made another upstream comment, so fingers crossed. This is a data loss bug too :(

Patches welcome.

[Phil Bull (philbull)]

OK, I can confirm the EPS issue but I can't reproduce the truncation issue.

[Matěj Laitl (f-matej)]

Version: 1.3.92 (using KDE KDE 3.5.3)
Installed from: Gentoo Packages
Compiler: GCC 4.1
OS: Linux

This bug applies to *SOME* JPEGs.
When I try rotate a jpeg image, it displays correctly.
But when I save it, a thin row of pixels (usually 5 to 10 pixels thick - depends on the image) is magically transfered from top to bottom. (or other directions)

If I rotate the original image, but save it in PNG instead of JPEG, the problem is gone.

This problem also aplies to Flip and Mirror operations.

This seems related to new JPEG lossles rotation capability introduced in 1.3.92

I'm gonna upload images where this occurs.

I checked it in GV versions: 1.3.92 and SVN revison 572689 (dated 2006-08-13)

[Matěj Laitl (f-matej)]

Created attachment 17356
Original Image

[Matěj Laitl (f-matej)]

Created attachment 17357
Rotated Image

Note the image only seems to be B/W. Technically it's full RGB.

PS: Like the girls? ;-)

[Danakil-kde (danakil-kde)]

Same problem here with this picture : http://danakil.free.fr/linux/bug_if_rotated.jpg
http://danakil.free.fr/linux/right_rotated.jpg -> bug

I don't know if this is important but this picture was resized with the "Resize images..." kipi plugin

[Matěj Laitl (f-matej)]

Still occurs with gwenview 1.4.0 using KDE 3.5.5

[Aurélien Gâteau (agateau)]

This is because the JPEG image dimensions are not a multiple of the MCU size see [1] for an explanation). In this case, there are three solutions to perform a rotation:

1: Apply a lossless rotation, keeping the odd pixels around. Rotating the image the other way will give you back the original image.

2: Apply a lossless rotation, but trim the offending pixels, in your case this would reduce the height of the rotated image from 900 to 896. Rotating the image the other way won't give you back the original image.

3: Apply a lossy rotation, which would give a correct image, but would probably loose some details.

For now, Gwenview adopts a conservative behavior and use #1. What do you think should be done? let the user choose the solution? select #2 or #3 without asking?

If you want to try #2, edit imageutils/jpegcontent.cpp, find the line that says:

  transformoption.trim = false;

and change it to:

  transformoption.trim = true;

[1]: http://www.impulseadventure.com/photo/jpeg-minimum-coded-unit.html

[Matěj Laitl (f-matej)]

Oh, thanks for the nice explanation...

Is it possible to apply a lossy rotation only to the partial MCUs on the sides, while keeping lossless for the rest? Looks like a good compromise for me.

Trimming an image a little (the worst case would be 15 pixels) is not IMHO transparent for user... It alters composition or user-made margins etc.

How often you rotate & save multiple times? I prefer small quality loss to erasing a small part of image.

(maybe stupid) idea: If it saved rotated JPEGs with slightly lower compression ratio, would it eliminate the quality loss?

[Matěj Laitl (f-matej)]

Oh, now I know that i can't be only partially lossy.

So the best solution IMHO:
offer 2 or 3 to an user, and add configuration options:
  always 2.
  always 3.
  ask before each save of rotated odd-sized jpeg.

what about the "maybe stupid idea"?

Now i will only resize my JPEGs to multiples of 8.

[Aurélien Gâteau (agateau)]

Thanks for your suggestions, I will think about it.

As of your "maybe stupid idea": I think it could compensate for some quality loss, but it won't eliminate it and would of course result in a larger file.

[Rich Johnson (nixternal)]

Still present in Gwenview 1.4.0 (KDE 3.5.5) in Kubuntu 7.04 (Feisty)

[Rich Johnson (nixternal)]

Just noticed a similar, if not same, bug on the KDE tracking system. In the case of the reporter it was with a JPEG image. I added my note concerning it also appears with .eps files as well.

Adding upstream bug tracking number for KDE

[Rich Johnson (nixternal)]

This same bug/issue also occurs for me with .eps files. A bug has been reported on the Ubuntu Bug Tracker (Malone) concerning this as well.

https://launchpad.net/distros/ubuntu/+source/gwenview/+bug/29627

Using:
 Gwenview 1.4.0
 KDE 3.5.5

[Rich Johnson (nixternal)]

wrong upstream bug originally reported.

[Posix-ru-w (posix-ru-w)]

Hi guys,

the same here. KDE 4.5, openSUSE 11.3.

> rpm -qa gwenview
gwenview-4.5.0-170.2.i586

What's about patch or something like this?

[BajK (kaiuwebroulik2)]

I found a solution for myself: Rotating the image, then cutting the out-of-place-part, flipping it (since it is upside down), and attaching it to the top. -> done :D
Still annoying.

[Alexander van Loon (avanloon)]

Any news on this? Because I just witnessed this bug occur with Gwenview 2.7.2 on KDE 4.7.3.

[Octavian Petre (octavsly)]

gwenview 2.6.4 on KDE 4.6.5 is OK

[Mark S. (invalidopcode)]

With Gwenview 2.7.4 on KDE 4.7.4, I can still reproduce this bug using Matěj Laitl's original pictures.

[Myriam Schweingruber (myriam)]

*** Bug 153066 has been marked as a duplicate of this bug. ***

[Aurélien Gâteau (agateau)]

*** Bug 273417 has been marked as a duplicate of this bug. ***

[Aurélien Gâteau (agateau)]

*** Bug 252693 has been marked as a duplicate of this bug. ***

[Martin Klapetek (martin-klapetek)]

Still present in 4.10.3.