Ubuntu
videoporama package

Imported images are not rotated based on exif meta data

Bug #511882 reported by Daniel Ellis
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
videoporama (Ubuntu)
Confirmed
Wishlist
Unassigned

Bug Description

Binary package hint: videoporama

When importing images from certain digital cameras, images which are taken as portraits appear on their side. You have to manually rotate each of the effected images using a program such as GIMP before importing them to videoporama.

I have a solution to this using python_pyexiv2 to discover the orientation of the images. However this will be an extra dependency on the project.

Daniel Ellis (danellisuk)
Changed in videoporama (Ubuntu):
status: New → In Progress
assignee: nobody → Daniel Ellis (danellisuk)
Revision history for this message
Alessio Treglia (quadrispro) wrote :

Great work, Daniel.

Is it ready to be uploaded?

Can you send your patch to upstream's author? [1]

[1] http://www.videoporama.tuxfamily.org/index.php?z=9

Changed in videoporama (Ubuntu):
importance: Undecided → Wishlist
Revision history for this message
Daniel Ellis (danellisuk) wrote :

Thanks, I have now sent the patch to Olivier (the author). The patch does work and there is an auto_rotate option in the config file which the default is set to 1. However I have not managed to update the configuration GUI to include this config option yet. However, I doubt there would be any need to disable the functionality for most people.

If this patch is to be included in the Ubuntu repository, it will need updating to **require** the package python_pyexiv2. If anyone knows how to make python continue if a certain import statement fails, then we could make videoporama just recommend python_pyexiv2 instead of requiring it.

Revision history for this message
Alessio Treglia (quadrispro) wrote : Re: [Bug 511882] Re: Imported images are not rotated based on exif meta data

On Fri, Jan 29, 2010 at 9:12 AM, Daniel Ellis <email address hidden> wrote:
> If this patch is to be included in the Ubuntu repository, it will need
> updating to **require** the package python_pyexiv2.  If anyone knows how
> to make python continue if a certain import statement fails, then we
> could make videoporama just recommend python_pyexiv2 instead of
> requiring it.
>

Of course you can wrap your import statements with try&except:

try:
    import pyexiv2
except ImportError:
    pass # alternatively you can also print a warning

BTW, thank you for sending patch to upstream!

--
Alessio Treglia <email address hidden>
Ubuntu MOTU Developer | Homepage: http://www.alessiotreglia.com
0FEC 59A5 E18E E04F 6D40 593B 45D4 8C7C DCFC 3FD0

Daniel Ellis (danellisuk)
Changed in videoporama (Ubuntu):
assignee: Daniel Ellis (danellisuk) → nobody
status: In Progress → Confirmed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.