[hugin-ptx] Re: Compiling lens calibrate
Am Thursday 10 September 2009 schrieb T. Modes: > > Hi Tim, > > I tried to run lens_calibrate. > But it crashed because there are no folder named "output". After > creating the output folder it run, but it did not check the output. > > Could you fix the code, so that > a) save the output on a other place (e. g. temp folder) > or b) check if the output folder exists and create it if necessary Ok, it should not crash. I think a warning were reasonable. But creating folder (which may well not be desired) can also make some trouble. > Thomas Kornel -- Kornel Benko kornel.be...@berlin.de signature.asc Description: This is a digitally signed message part.
[hugin-ptx] Re: Compiling lens calibrate
Hi Tim, I tried to run lens_calibrate. But it crashed because there are no folder named "output". After creating the output folder it run, but it did not check the output. Could you fix the code, so that a) save the output on a other place (e. g. temp folder) or b) check if the output folder exists and create it if necessary Thomas --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "hugin and other free panoramic software" group. A list of frequently asked questions is available at: http://wiki.panotools.org/Hugin_FAQ To post to this group, send email to hugin-ptx@googlegroups.com To unsubscribe from this group, send email to hugin-ptx-unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/hugin-ptx -~--~~~~--~~--~--~---
[hugin-ptx] Re: Compiling lens calibrate
Yep I just realised that and removed it, I don't think CMakeLists.txt need license info (none of the others seem to have it). Cheers, Tim 2009/9/5 T. Modes > > Tim schrieb: > > Ok I removed the jhead stuff, was used to check for buggy canon 20D > > data apparently: > > > > Thanks Tim, > now it compiles. > > But the licence header in lens_calibrate/CMakeLists.txt makes > problems. I think CMake is using a # for comments, the c-style > comment /* */ does not work (CMake is complaining about not finding > command /*…). > > Thomas > > > --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "hugin and other free panoramic software" group. A list of frequently asked questions is available at: http://wiki.panotools.org/Hugin_FAQ To post to this group, send email to hugin-ptx@googlegroups.com To unsubscribe from this group, send email to hugin-ptx-unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/hugin-ptx -~--~~~~--~~--~--~---
[hugin-ptx] Re: Compiling lens calibrate
Tim schrieb: > Ok I removed the jhead stuff, was used to check for buggy canon 20D > data apparently: > Thanks Tim, now it compiles. But the licence header in lens_calibrate/CMakeLists.txt makes problems. I think CMake is using a # for comments, the c-style comment /* */ does not work (CMake is complaining about not finding command /*…). Thomas --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "hugin and other free panoramic software" group. A list of frequently asked questions is available at: http://wiki.panotools.org/Hugin_FAQ To post to this group, send email to hugin-ptx@googlegroups.com To unsubscribe from this group, send email to hugin-ptx-unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/hugin-ptx -~--~~~~--~~--~--~---
[hugin-ptx] Re: Compiling lens calibrate
Ok I removed the jhead stuff, was used to check for buggy canon 20D
data apparently:
if (getExifModel() == "Canon EOS 20D") {
// special case for buggy 20D camera
sensorSize.x = 22.5;
sensorSize.y = 15;
}
I'll see if I can do the same check with exiv2.
Cheers,
Tim
On 5 Sep, 11:38, Tim wrote:
> Compiles on Tom's (Sharpless) system - maybe he has it set up
> differently
>
> The code actually uses jhead and exiv2 (as does SrcPanoImage.cpp).
>
> I'll investigate removing the jhead stuff.
>
> Tim
>
> On 5 Sep, 11:31, Bruno Postle wrote:
>
> > On Sat 05-Sep-2009 at 02:00 -0700, T. Modes wrote:
>
> > >lens_calibrate is using jhead lib, but his library was removed from
> > >hugin.
>
> > We could reinclude jhead, but it was a very old version that Pablo
> > turned into a library.
>
> > The rest of hugin switched to exiv2 as jhead only supports JPEG, so
> > really lens_calibrate needs to be fixed to use exiv2.
>
> > >Furthermore lens_calibrate needs to add to CMakeLists.txt in src (see
> > >patch):
>
> > This needs to be applied, but obviously it breaks the build at the
> > moment.
>
> > --
> > Bruno
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups
"hugin and other free panoramic software" group.
A list of frequently asked questions is available at:
http://wiki.panotools.org/Hugin_FAQ
To post to this group, send email to hugin-ptx@googlegroups.com
To unsubscribe from this group, send email to
hugin-ptx-unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/hugin-ptx
-~--~~~~--~~--~--~---
[hugin-ptx] Re: Compiling lens calibrate
Compiles on Tom's (Sharpless) system - maybe he has it set up differently The code actually uses jhead and exiv2 (as does SrcPanoImage.cpp). I'll investigate removing the jhead stuff. Tim On 5 Sep, 11:31, Bruno Postle wrote: > On Sat 05-Sep-2009 at 02:00 -0700, T. Modes wrote: > > > > >lens_calibrate is using jhead lib, but his library was removed from > >hugin. > > We could reinclude jhead, but it was a very old version that Pablo > turned into a library. > > The rest of hugin switched to exiv2 as jhead only supports JPEG, so > really lens_calibrate needs to be fixed to use exiv2. > > >Furthermore lens_calibrate needs to add to CMakeLists.txt in src (see > >patch): > > This needs to be applied, but obviously it breaks the build at the > moment. > > -- > Bruno --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "hugin and other free panoramic software" group. A list of frequently asked questions is available at: http://wiki.panotools.org/Hugin_FAQ To post to this group, send email to hugin-ptx@googlegroups.com To unsubscribe from this group, send email to hugin-ptx-unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/hugin-ptx -~--~~~~--~~--~--~---
[hugin-ptx] Re: Compiling lens calibrate
On Sat 05-Sep-2009 at 02:00 -0700, T. Modes wrote: > >lens_calibrate is using jhead lib, but his library was removed from >hugin. We could reinclude jhead, but it was a very old version that Pablo turned into a library. The rest of hugin switched to exiv2 as jhead only supports JPEG, so really lens_calibrate needs to be fixed to use exiv2. >Furthermore lens_calibrate needs to add to CMakeLists.txt in src (see >patch): This needs to be applied, but obviously it breaks the build at the moment. -- Bruno --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "hugin and other free panoramic software" group. A list of frequently asked questions is available at: http://wiki.panotools.org/Hugin_FAQ To post to this group, send email to hugin-ptx@googlegroups.com To unsubscribe from this group, send email to hugin-ptx-unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/hugin-ptx -~--~~~~--~~--~--~---
