hugin-devs team mailing list archive
-
hugin-devs team
-
Mailing list archive
-
Message #05918
Re: [Bug 679654] Re: updated levmar library
Hi Thomas
Thank you very much for your explanations. I will try to add the lapack
dependency to the Ubuntu packages I maintain and to the upstream Debian
packages as well. However, this effort will be concentrate on
hugin>2015.0.0 because I do not want to introduce a new dependency
(and, maybe, a new behavior and new problem spots) before the upcoming
2015.0.0 release.
Again, thank you very much for your advice.
With kind regards
Stefan
On 24.06.2015 20:10, tmodes wrote:
> Hi Stephan,
>
> there are 2 type of warnings in levmar.
>
> The first one is of type
> #warning Linearly constrained optimization requires LAPACK and was not compiled!
>
> levmar implement different unconstrained and constrained versions of the
> Levenberg-Marquard algorithm. Some of the contrained version need LAPACK
> to work. But Hugin is only using the unconstrained version. This type of
> warning can therefore ignored, at least for Hugin.
>
> The second warning is
> #warning LAPACK not available, LU will be used for matrix inversion when computing the covariance; this might be unstable at times
>
> This warning is trigged in code which is used by Hugin. The warning
> states that LAPACK could help to calculate the covariance. But as far as
> I know Hugin is not using the returned covariance. So LAPACK could help
> to make it more robust. But until now I have not seen any issues when
> using levmar without LAPACK.
>
> The LAPACK is already in CMake, but needs explicitly activated:
> ##
> ## LAPACK (optional, enable by -DENABLE_LAPACK=ON)
> ##
> OPTION (ENABLE_LAPACK "Use LAPACK based solver in levmar" OFF)
>
--
Any technology that does not appear magical is insufficiently advanced.
~ Gregory Benford
--
You received this bug notification because you are a member of Hugin
Developers, which is subscribed to Hugin.
https://bugs.launchpad.net/bugs/679654
Title:
updated levmar library
Status in Hugin - Panorama Tools GUI:
Fix Committed
Bug description:
Last December, an updated levmar was released (2.2), with change that
could be very interesting for hugin:
- Fixed an error in the computation of the damping factor of
levmar_dif() in lm_core.c. Thanks to John Ervin for reporting.
This change is interesting, because levmar_dif is used for photometric
optimisation by hugin.
I took the liberty to try hugin with this newer version of the levmar
library, and on my test panorama it took slightly longer but reached
the same end result.
This is of course not 0.7 material, but perhaps something for early
0.7.1.
To manage notifications about this bug go to:
https://bugs.launchpad.net/hugin/+bug/679654/+subscriptions
References