[hugin-ptx] Re: Compiling lens calibrate

2009-09-09 Thread 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

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

2009-09-05 Thread Bruno Postle

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

2009-09-05 Thread Tim

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 br...@postle.net 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

2009-09-05 Thread Tim

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 timnug...@gmail.com 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 br...@postle.net 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

2009-09-05 Thread 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

2009-09-05 Thread Tim Nugent
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 thomas.mo...@gmx.de


 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
-~--~~~~--~~--~--~---