Better exception handling for deghosting tools

Bug #679718 reported by Iouri Ivliev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Hugin
Confirmed
Medium
Lukas Jirkovsky

Bug Description

what() function in BadDimensions and NoImages classes should be 'const' to override std::exception's what() properly.
patch-src-tools-hugin_hdrmerge.cpp.diff contains also quick&dirty hack to restore alpha channel in image produced by khan algorithm. This isn't "Right Way", but allows to use combined image stacks with enblend. Just IMHO, Deghosting class shouldn't check input images dimensions in loadImages() functions. An overlapping rectangle should be calculated using input images bitmap sizes, page offsets and alpha channels. Sizes and offsets of this rectangle should be used as sizes and offsets of output image. Alpha channel of output image should be made as intersection or joining of input images alpha channels.

Tags: patch
Revision history for this message
Iouri Ivliev (lp-0o6vs1) wrote :

The file patch-deghosting.tar.gz was added: None

Revision history for this message
Lukas Jirkovsky (l-jirkovsky) wrote :

Thank you very much for the patches and suggestions. I've committed the ones which fixes exceptions.

I'll take a look at the alpha handling and computation of overlapping region later.

Revision history for this message
Yuv (yuv) wrote :

yes, thank you. would you mind sharing your name with us so we can add you to the credits page of Hugin?

Revision history for this message
Iouri Ivliev (lp-0o6vs1) wrote :

My name is Iouri Ivliev

Revision history for this message
Iouri Ivliev (lp-0o6vs1) wrote :

The file merge-wrapper.gz was added: Simple shell script wrapper to hugin_hdrmerge

Revision history for this message
Iouri Ivliev (lp-0o6vs1) wrote :

To: Lukas Jirkovsky
BTW, look at attached merge-wrapper.gz. This is gzipped shell script wrapper to hugin_hdrmerge. It calculates overlapping rectangle geometry as intersection of input images geometry and composes output alpha channel as intersection of input images alpha channels.

Yuv (yuv)
tags: added: patch
Changed in hugin:
assignee: nobody → Lukas Jirkovsky (l-jirkovsky)
status: New → Triaged
Changed in hugin:
importance: High → Medium
status: Triaged → Confirmed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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