[Bug 197619] New: Make inverse perspective transform not crop images

classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 197619] New: Make inverse perspective transform not crop images

Bugzilla from b.traut@gmx.net
https://bugs.kde.org/show_bug.cgi?id=197619

           Summary: Make inverse perspective transform not crop images
           Product: digikamimageplugins
           Version: unspecified
          Platform: Compiled Sources
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: wishlist
          Priority: NOR
         Component: Perspective
        AssignedTo: [hidden email]
        ReportedBy: [hidden email]


Version:            (using Devel)
OS:                Linux
Installed from:    Compiled sources

Currently the inverse perspective transform of the perspective adjustment tool
crops the image to the selected rectangle.
This makes it less usefull for correcting photographs.
I would like to see at least two options here:

[X] Inverse transformation
  (o) Do not crop
  ( ) Crop to selected rectagle
  ( ) Crop to valid part (<-- this might not be well definable, since there are
many ways of cropping the resulting image)

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 197619] Make inverse perspective transform not crop images

Bugzilla from b.traut@gmx.net
https://bugs.kde.org/show_bug.cgi?id=197619


Benjamin Traut <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]




--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 197619] Make inverse perspective transform not crop images

Gilles Caulier-4
In reply to this post by Bugzilla from b.traut@gmx.net
https://bugs.kde.org/show_bug.cgi?id=197619


Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|unspecified                 |1.0.0




--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 197619] Make inverse perspective transform not crop images

Gilles Caulier-4
In reply to this post by Bugzilla from b.traut@gmx.net
https://bugs.kde.org/show_bug.cgi?id=197619


Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |david.nolden.kde@art-master
                   |                            |.de




--- Comment #1 from Gilles Caulier <caulier gilles gmail com>  2009-06-23 16:16:00 ---
David,

What do you think about this entry ?

Gilles Caulier

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 197619] Make inverse perspective transform not crop images

Bugzilla from david.nolden.kde@art-master.de
In reply to this post by Bugzilla from b.traut@gmx.net
https://bugs.kde.org/show_bug.cgi?id=197619





--- Comment #2 from David Nolden <david nolden kde art-master de>  2009-06-23 17:19:29 ---
What would "crop to valid parts" mean? Otherwise, yes I think that the first
two options would be useful.

Although to be really correct, the image size would even have to be increased
in some cases, like the rotation tool does.

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 197619] Make inverse perspective transform not crop images

Bugzilla from b.traut@gmx.net
In reply to this post by Bugzilla from b.traut@gmx.net
https://bugs.kde.org/show_bug.cgi?id=197619





--- Comment #3 from Benjamin Traut <b traut gmx net>  2009-06-23 17:39:42 ---
(In reply to comment #2)
> What would "crop to valid parts" mean? Otherwise, yes I think that the first
> two options would be useful.

When you apply a perspective transformation to an image, you get black parts
where there weren't any pixels which were transformed.
With "crop to valid parts" I meant, that the algorithm should try to crop to a
rectangle which lies inside the image and does not cover any of those invalid,
black areas.
Since there are many ways of defining such a rectangle, it might be usefull to
have this algorithm make an initial guess (e.g. make a valid rectangle which
resembles the aspect ratio of the image) that could be used as an input for the
crop tool.
Hence "this might not be well definable, since there are many ways of cropping
the resulting image".
To sum it up, I really do not know whether option 3 is an improvement or not.

> Although to be really correct, the image size would even have to be increased
> in some cases, like the rotation tool does.

Yes, the size of the image has to be recalculated.

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 197619] Make inverse perspective transform not crop images

Bugzilla from mikmach@wp.pl
In reply to this post by Bugzilla from b.traut@gmx.net
https://bugs.kde.org/show_bug.cgi?id=197619





--- Comment #4 from Mikolaj Machowski <mikmach wp pl>  2009-06-23 18:19:36 ---
Another solution: Copy interface from Free rotation - identical widget, similar
results, less thinking for user which is always good thing.

In Free Rotation there is drop-down menu for Auto-crop option: None, Widest
Area, Largest Area. Additional option - Highest area  - does it make sense?

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 197619] Make inverse perspective transform not crop images

bugzilla_noreply
In reply to this post by Bugzilla from b.traut@gmx.net
https://bugs.kde.org/show_bug.cgi?id=197619

[hidden email] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|Perspective                 |Tool-Perspective
            Product|digikamimageplugins         |digikam

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel