[Hugin-bug-hunters] [Bug 2053287] Re: sometimes there are black patches/corners in blended images
@Bruno Since last autumn (~September) main line vigra should support c++17, but any distribution release is of course likely to not have included that yet. But I guess it will fix itself in Fedora some time. (wrt msvc, I was referring to what Thomas wrote.) -- You received this bug notification because you are a member of Hugin Bug Hunters, which is subscribed to Enblend. https://bugs.launchpad.net/bugs/2053287 Title: sometimes there are black patches/corners in blended images Status in Enblend: Fix Committed Bug description: As has been known for a long time, enblend sometimes produces black patches in the corners of subimages. As we have discussed by email, I'd like to suggest a patch for that: -- patch1 contains the pyramid type to make the following simpler -- patch2 contains the actual change where the mask is corrected using both alpha layers prior to the actual blending step -- patch3 are two small functional changes that are unrelated to the main issue -- patch4 are modernisation and style improvements, that I just had to do while working on the code. If you think some of them don't add enough (or any) value, feel free to drop some. cheers, lukas wirz To manage notifications about this bug go to: https://bugs.launchpad.net/enblend/+bug/2053287/+subscriptions ___ Mailing list: https://launchpad.net/~hugin-bug-hunters Post to : hugin-bug-hunters@lists.launchpad.net Unsubscribe : https://launchpad.net/~hugin-bug-hunters More help : https://help.launchpad.net/ListHelp
[Hugin-bug-hunters] [Bug 2053287] Re: sometimes there are black patches/corners in blended images
I'm not using msvc. This error is with fedora, but it looks like vigra in Ubuntu has already been patched to support c++17. The fedora vigra maintainer needs to copy some Ubuntu patches. -- You received this bug notification because you are a member of Hugin Bug Hunters, which is subscribed to Enblend. https://bugs.launchpad.net/bugs/2053287 Title: sometimes there are black patches/corners in blended images Status in Enblend: Fix Committed Bug description: As has been known for a long time, enblend sometimes produces black patches in the corners of subimages. As we have discussed by email, I'd like to suggest a patch for that: -- patch1 contains the pyramid type to make the following simpler -- patch2 contains the actual change where the mask is corrected using both alpha layers prior to the actual blending step -- patch3 are two small functional changes that are unrelated to the main issue -- patch4 are modernisation and style improvements, that I just had to do while working on the code. If you think some of them don't add enough (or any) value, feel free to drop some. cheers, lukas wirz To manage notifications about this bug go to: https://bugs.launchpad.net/enblend/+bug/2053287/+subscriptions ___ Mailing list: https://launchpad.net/~hugin-bug-hunters Post to : hugin-bug-hunters@lists.launchpad.net Unsubscribe : https://launchpad.net/~hugin-bug-hunters More help : https://help.launchpad.net/ListHelp
[Hugin-bug-hunters] [Bug 721136] Re: enblend creates an unexplainable black area.
The bug was what we had discussed on the hugin mailing list four years ago: https://groups.google.com/g/hugin-ptx/c/IgglWhsUCEQ/m/6bUu_YQpCAAJ tldr: one may generate a seam line such that pixels are included from an image that are out of bounds wrt that image. This fix only affects the blending stage. The seam generation and optimisation are off in more subtly ways, and harder to fix. -- You received this bug notification because you are a member of Hugin Bug Hunters, which is subscribed to Enblend. https://bugs.launchpad.net/bugs/721136 Title: enblend creates an unexplainable black area. Status in Enblend: Fix Committed Bug description: enblending http://prive.bitwizard.nl/t80001.tif and http://prive.bitwizard.nl/t80003.tif results in http://prive.bitwizard.nl/t8a.tif I cannot say I expected that black triangle in the output. My enblend was pulled from hg this morning (no changes) and built without imagecache or openMP. (imagecache causes corruption of the output similar but different to this. ) To manage notifications about this bug go to: https://bugs.launchpad.net/enblend/+bug/721136/+subscriptions ___ Mailing list: https://launchpad.net/~hugin-bug-hunters Post to : hugin-bug-hunters@lists.launchpad.net Unsubscribe : https://launchpad.net/~hugin-bug-hunters More help : https://help.launchpad.net/ListHelp
[Hugin-bug-hunters] [Bug 2053287] Re: sometimes there are black patches/corners in blended images
Thank you for merging and testing! The part that you omitted wasn't very important, but I am surprised that it causes issues with msvc (I had not tested that). I am not surprised, but slightly relieved that not only my but also your test cases pass! -- You received this bug notification because you are a member of Hugin Bug Hunters, which is subscribed to Enblend. https://bugs.launchpad.net/bugs/2053287 Title: sometimes there are black patches/corners in blended images Status in Enblend: Fix Committed Bug description: As has been known for a long time, enblend sometimes produces black patches in the corners of subimages. As we have discussed by email, I'd like to suggest a patch for that: -- patch1 contains the pyramid type to make the following simpler -- patch2 contains the actual change where the mask is corrected using both alpha layers prior to the actual blending step -- patch3 are two small functional changes that are unrelated to the main issue -- patch4 are modernisation and style improvements, that I just had to do while working on the code. If you think some of them don't add enough (or any) value, feel free to drop some. cheers, lukas wirz To manage notifications about this bug go to: https://bugs.launchpad.net/enblend/+bug/2053287/+subscriptions ___ Mailing list: https://launchpad.net/~hugin-bug-hunters Post to : hugin-bug-hunters@lists.launchpad.net Unsubscribe : https://launchpad.net/~hugin-bug-hunters More help : https://help.launchpad.net/ListHelp
[Hugin-bug-hunters] [Bug 721136] Re: enblend creates an unexplainable black area.
Thanks so much for fixing this bug. What was the underlying problem? -- You received this bug notification because you are a member of Hugin Bug Hunters, which is subscribed to Enblend. https://bugs.launchpad.net/bugs/721136 Title: enblend creates an unexplainable black area. Status in Enblend: Fix Committed Bug description: enblending http://prive.bitwizard.nl/t80001.tif and http://prive.bitwizard.nl/t80003.tif results in http://prive.bitwizard.nl/t8a.tif I cannot say I expected that black triangle in the output. My enblend was pulled from hg this morning (no changes) and built without imagecache or openMP. (imagecache causes corruption of the output similar but different to this. ) To manage notifications about this bug go to: https://bugs.launchpad.net/enblend/+bug/721136/+subscriptions ___ Mailing list: https://launchpad.net/~hugin-bug-hunters Post to : hugin-bug-hunters@lists.launchpad.net Unsubscribe : https://launchpad.net/~hugin-bug-hunters More help : https://help.launchpad.net/ListHelp
[Hugin-bug-hunters] [Bug 785803] Re: Enblend --no-optimize produces black holes into panoramas
Fixes with changes in #2053287 ** Changed in: enblend Status: Confirmed => Fix Committed -- You received this bug notification because you are a member of Hugin Bug Hunters, which is subscribed to Enblend. https://bugs.launchpad.net/bugs/785803 Title: Enblend --no-optimize produces black holes into panoramas Status in Enblend: Fix Committed Bug description: --no-optimize: http://i.imgur.com/Bc9hD.jpg How it should look: http://i.imgur.com/MAyhz.jpg Win7 x64 w/ 32bit and x64 Hugin 2011.1 RC1 (2011.0.0.8f1447ab8649 built by Matthew Petroff), multithreaded Enblend. Project contains less than ten fisheye images encompassing the whole panosphere, so it's your average spherical panorama. You can see how much the images overlap here: http://i.imgur.com/KuAqU.jpg I think the issue could be related to too small overlap between images. To manage notifications about this bug go to: https://bugs.launchpad.net/enblend/+bug/785803/+subscriptions ___ Mailing list: https://launchpad.net/~hugin-bug-hunters Post to : hugin-bug-hunters@lists.launchpad.net Unsubscribe : https://launchpad.net/~hugin-bug-hunters More help : https://help.launchpad.net/ListHelp
[Hugin-bug-hunters] [Bug 721136] Re: enblend creates an unexplainable black area.
Fixes with changes in #2053287 ** Changed in: enblend Status: Confirmed => Fix Committed -- You received this bug notification because you are a member of Hugin Bug Hunters, which is subscribed to Enblend. https://bugs.launchpad.net/bugs/721136 Title: enblend creates an unexplainable black area. Status in Enblend: Fix Committed Bug description: enblending http://prive.bitwizard.nl/t80001.tif and http://prive.bitwizard.nl/t80003.tif results in http://prive.bitwizard.nl/t8a.tif I cannot say I expected that black triangle in the output. My enblend was pulled from hg this morning (no changes) and built without imagecache or openMP. (imagecache causes corruption of the output similar but different to this. ) To manage notifications about this bug go to: https://bugs.launchpad.net/enblend/+bug/721136/+subscriptions ___ Mailing list: https://launchpad.net/~hugin-bug-hunters Post to : hugin-bug-hunters@lists.launchpad.net Unsubscribe : https://launchpad.net/~hugin-bug-hunters More help : https://help.launchpad.net/ListHelp
[Hugin-bug-hunters] [Bug 2053287] Re: sometimes there are black patches/corners in blended images
Thanks for testing. I also closed the mentioned tickets. -- You received this bug notification because you are a member of Hugin Bug Hunters, which is subscribed to Enblend. https://bugs.launchpad.net/bugs/2053287 Title: sometimes there are black patches/corners in blended images Status in Enblend: Fix Committed Bug description: As has been known for a long time, enblend sometimes produces black patches in the corners of subimages. As we have discussed by email, I'd like to suggest a patch for that: -- patch1 contains the pyramid type to make the following simpler -- patch2 contains the actual change where the mask is corrected using both alpha layers prior to the actual blending step -- patch3 are two small functional changes that are unrelated to the main issue -- patch4 are modernisation and style improvements, that I just had to do while working on the code. If you think some of them don't add enough (or any) value, feel free to drop some. cheers, lukas wirz To manage notifications about this bug go to: https://bugs.launchpad.net/enblend/+bug/2053287/+subscriptions ___ Mailing list: https://launchpad.net/~hugin-bug-hunters Post to : hugin-bug-hunters@lists.launchpad.net Unsubscribe : https://launchpad.net/~hugin-bug-hunters More help : https://help.launchpad.net/ListHelp
[Hugin-bug-hunters] [Bug 2053287] Re: sometimes there are black patches/corners in blended images
It also passes the test case I uploaded here: https://bugs.launchpad.net/enblend/+bug/785803/comments/8 -- You received this bug notification because you are a member of Hugin Bug Hunters, which is subscribed to Enblend. https://bugs.launchpad.net/bugs/2053287 Title: sometimes there are black patches/corners in blended images Status in Enblend: Fix Committed Bug description: As has been known for a long time, enblend sometimes produces black patches in the corners of subimages. As we have discussed by email, I'd like to suggest a patch for that: -- patch1 contains the pyramid type to make the following simpler -- patch2 contains the actual change where the mask is corrected using both alpha layers prior to the actual blending step -- patch3 are two small functional changes that are unrelated to the main issue -- patch4 are modernisation and style improvements, that I just had to do while working on the code. If you think some of them don't add enough (or any) value, feel free to drop some. cheers, lukas wirz To manage notifications about this bug go to: https://bugs.launchpad.net/enblend/+bug/2053287/+subscriptions ___ Mailing list: https://launchpad.net/~hugin-bug-hunters Post to : hugin-bug-hunters@lists.launchpad.net Unsubscribe : https://launchpad.net/~hugin-bug-hunters More help : https://help.launchpad.net/ListHelp
[Hugin-bug-hunters] [Bug 2053287] Re: sometimes there are black patches/corners in blended images
Can confirm that the patched enblend passes all the tests that I uploaded here: https://bugs.launchpad.net/enblend/+bug/721136/comments/18 -- You received this bug notification because you are a member of Hugin Bug Hunters, which is subscribed to Enblend. https://bugs.launchpad.net/bugs/2053287 Title: sometimes there are black patches/corners in blended images Status in Enblend: Fix Committed Bug description: As has been known for a long time, enblend sometimes produces black patches in the corners of subimages. As we have discussed by email, I'd like to suggest a patch for that: -- patch1 contains the pyramid type to make the following simpler -- patch2 contains the actual change where the mask is corrected using both alpha layers prior to the actual blending step -- patch3 are two small functional changes that are unrelated to the main issue -- patch4 are modernisation and style improvements, that I just had to do while working on the code. If you think some of them don't add enough (or any) value, feel free to drop some. cheers, lukas wirz To manage notifications about this bug go to: https://bugs.launchpad.net/enblend/+bug/2053287/+subscriptions ___ Mailing list: https://launchpad.net/~hugin-bug-hunters Post to : hugin-bug-hunters@lists.launchpad.net Unsubscribe : https://launchpad.net/~hugin-bug-hunters More help : https://help.launchpad.net/ListHelp
[Hugin-bug-hunters] [Bug 2053287] Re: sometimes there are black patches/corners in blended images
Yes, I replaced the `throw(PreconditionViolation)` with `noexcept(false)` in both vigra locations and now enblend builds. I'm using enblend cmake, as the autoconf stuff needed bootstrapping and I couldn't remember how to do it. Haven't tested the bugfix yet, I seem to remember I created test cases with several examples that all failed at some time. -- You received this bug notification because you are a member of Hugin Bug Hunters, which is subscribed to Enblend. https://bugs.launchpad.net/bugs/2053287 Title: sometimes there are black patches/corners in blended images Status in Enblend: Fix Committed Bug description: As has been known for a long time, enblend sometimes produces black patches in the corners of subimages. As we have discussed by email, I'd like to suggest a patch for that: -- patch1 contains the pyramid type to make the following simpler -- patch2 contains the actual change where the mask is corrected using both alpha layers prior to the actual blending step -- patch3 are two small functional changes that are unrelated to the main issue -- patch4 are modernisation and style improvements, that I just had to do while working on the code. If you think some of them don't add enough (or any) value, feel free to drop some. cheers, lukas wirz To manage notifications about this bug go to: https://bugs.launchpad.net/enblend/+bug/2053287/+subscriptions ___ Mailing list: https://launchpad.net/~hugin-bug-hunters Post to : hugin-bug-hunters@lists.launchpad.net Unsubscribe : https://launchpad.net/~hugin-bug-hunters More help : https://help.launchpad.net/ListHelp
[Hugin-bug-hunters] [Bug 2053287] Re: sometimes there are black patches/corners in blended images
@Bruno: Are you building with automake or with CMake? (On Ubuntu with CMake it works. But it seems they are providing a patched vigra library, where this offending line is missing.) @Lukas: I applied now most of your patches and pushed the changes to the repository. Thank you again for the patches. Some parts I skipped because of cosmetic issues or some part results in compile error with MSVC. ** Changed in: enblend Status: New => Fix Committed -- You received this bug notification because you are a member of Hugin Bug Hunters, which is subscribed to Enblend. https://bugs.launchpad.net/bugs/2053287 Title: sometimes there are black patches/corners in blended images Status in Enblend: Fix Committed Bug description: As has been known for a long time, enblend sometimes produces black patches in the corners of subimages. As we have discussed by email, I'd like to suggest a patch for that: -- patch1 contains the pyramid type to make the following simpler -- patch2 contains the actual change where the mask is corrected using both alpha layers prior to the actual blending step -- patch3 are two small functional changes that are unrelated to the main issue -- patch4 are modernisation and style improvements, that I just had to do while working on the code. If you think some of them don't add enough (or any) value, feel free to drop some. cheers, lukas wirz To manage notifications about this bug go to: https://bugs.launchpad.net/enblend/+bug/2053287/+subscriptions ___ Mailing list: https://launchpad.net/~hugin-bug-hunters Post to : hugin-bug-hunters@lists.launchpad.net Unsubscribe : https://launchpad.net/~hugin-bug-hunters More help : https://help.launchpad.net/ListHelp
[Hugin-bug-hunters] [Bug 2053287] Re: sometimes there are black patches/corners in blended images
Yes, that is weird. You're quoting vigra code, ... #if _MSC_VER >= 1900 || __cplusplus >= 201103L noexcept(false) #else throw(PreconditionViolation) #endif ... and given that enblend uses c++17, the offending line shouldn't be used. Does that problem not occur without my patches? Do you run some modified version of vigra? -- You received this bug notification because you are a member of Hugin Bug Hunters, which is subscribed to Enblend. https://bugs.launchpad.net/bugs/2053287 Title: sometimes there are black patches/corners in blended images Status in Enblend: New Bug description: As has been known for a long time, enblend sometimes produces black patches in the corners of subimages. As we have discussed by email, I'd like to suggest a patch for that: -- patch1 contains the pyramid type to make the following simpler -- patch2 contains the actual change where the mask is corrected using both alpha layers prior to the actual blending step -- patch3 are two small functional changes that are unrelated to the main issue -- patch4 are modernisation and style improvements, that I just had to do while working on the code. If you think some of them don't add enough (or any) value, feel free to drop some. cheers, lukas wirz To manage notifications about this bug go to: https://bugs.launchpad.net/enblend/+bug/2053287/+subscriptions ___ Mailing list: https://launchpad.net/~hugin-bug-hunters Post to : hugin-bug-hunters@lists.launchpad.net Unsubscribe : https://launchpad.net/~hugin-bug-hunters More help : https://help.launchpad.net/ListHelp
[Hugin-bug-hunters] [Bug 2053287] Re: sometimes there are black patches/corners in blended images
I can't get the latest mercurial with patches to build, this doesn't really make sense to me as enblend requires c++17: ``` /usr/include/vigra/separableconvolution.hxx:1413:13: error: ISO C++17 does not allow dynamic exception specifications 1413 | throw(PreconditionViolation) | ^ /usr/include/vigra/stdconvolution.hxx:796:13: error: ISO C++17 does not allow dynamic exception specifications 796 | throw(PreconditionViolation) | ^ ``` -- You received this bug notification because you are a member of Hugin Bug Hunters, which is subscribed to Enblend. https://bugs.launchpad.net/bugs/2053287 Title: sometimes there are black patches/corners in blended images Status in Enblend: New Bug description: As has been known for a long time, enblend sometimes produces black patches in the corners of subimages. As we have discussed by email, I'd like to suggest a patch for that: -- patch1 contains the pyramid type to make the following simpler -- patch2 contains the actual change where the mask is corrected using both alpha layers prior to the actual blending step -- patch3 are two small functional changes that are unrelated to the main issue -- patch4 are modernisation and style improvements, that I just had to do while working on the code. If you think some of them don't add enough (or any) value, feel free to drop some. cheers, lukas wirz To manage notifications about this bug go to: https://bugs.launchpad.net/enblend/+bug/2053287/+subscriptions ___ Mailing list: https://launchpad.net/~hugin-bug-hunters Post to : hugin-bug-hunters@lists.launchpad.net Unsubscribe : https://launchpad.net/~hugin-bug-hunters More help : https://help.launchpad.net/ListHelp