[hugin-ptx] Re: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
i don't understand where are CMakeLists.txt files. i've followed install instructions found here : http://wiki.panotools.org/Hugin_Compiling_Ubuntu#Pan-o-matic and i have no file to patch. could you please help me ? my error is : Making install in zthread/src make[1]: entrant dans le répertoire « /home/eighter/src/panomatic/panomatic-0.9.4/zthread/src » Making install in . make[2]: entrant dans le répertoire « /home/eighter/src/panomatic/panomatic-0.9.4/zthread/src » g++ -DPACKAGE_NAME=\"panomatic\" -DPACKAGE_TARNAME=\"panomatic\" -DPACKAGE_VERSION=\"1.0\" -DPACKAGE_STRING=\"panomatic\ 1.0\" -DPACKAGE_BUGREPORT=\"nao...@naouel.org\" -DPACKAGE=\"panomatic\" -DVERSION=\"1.0\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_DIRENT_H=1 -DSTDC_HEADERS=1 -DHAVE_LIMITS_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_UNISTD_H=1 -DHAVE_STDBOOL_H=1 -DHAVE_POSIX_THREADS= -DHAVE_SCHED_RT= -DHAVE_SCHED_YIELD= -DHAVE_PTHREAD_YIELD= -DHAVE_PTHREADKEY_CREATE= -DHAVE_BOOST= -DPNG_NO_ASSEMBLER_CODE= -I. -I../include -DNDEBUG -I/usr/include -L/usr/lib -MT Condition.o -MD -MP -MF .deps/Condition.Tpo -c -o Condition.o Condition.cxx In file included from ConditionImpl.h:26:0, from Condition.cxx:24: ../include/zthread/Guard.h: In instantiation of ‘ZThread::Guard::Guard(ZThread::Guard&) [with U = ZThread::FastLock; V = ZThread::LockedScope; LockType = ZThread::FastLock; LockingPolicy = ZThread::UnlockedScope]’: ConditionImpl.h:235:45: required from ‘void ZThread::ConditionImpl::wait() [with List = ZThread::fifo_list]’ Condition.cxx:52:17: required from here ../include/zthread/Guard.h:431:5: erreur: ‘extract’ was not declared in this scope, and no declarations were found by argument-dependent lookup at the point of instantiation [-fpermissive] ../include/zthread/Guard.h:431:5: note: declarations in dependent base ‘ZThread::LockHolder’ are not found by unqualified lookup ../include/zthread/Guard.h:431:5: note: use ‘this->extract’ instead make[2]: *** [Condition.o] Erreur 1 make[2]: quittant le répertoire « /home/eighter/src/panomatic/panomatic-0.9.4/zthread/src » make[1]: *** [install-recursive] Erreur 1 make[1]: quittant le répertoire « /home/eighter/src/panomatic/panomatic-0.9.4/zthread/src » make: *** [install-recursive] Erreur 1 eighter@eighter-Studio:~/src/panomatic/panomatic-0.9.4$ thanks in advance, regards, h. Le mercredi 13 janvier 2010 12:55:58 UTC+1, Harry van der Wolf a écrit : > > All, > > After Pablo's announcement[0] (3 January) of the almost patent free CP > detector I built it on OSX, but due to other priorities I had set myself, I > did not have the time to do tests (sorry Pablo [1]). > > This is only a mail to share some first test results. I need to do more > tests, so don't expect graphical and statistical test results. > == This mail is also to "invite" other users to test as well == > (but I have to admit that I did not create a portable plugin binary yet to > be delivered with Hugin for OSX). > > (My) General conclusion: Based on this prelimiary results and even as it > is now (but that was Pablo's own comment in his mail[0]), I think it does a > good job and I think it is suitable for use with Hugin but in combination > with celeste and cpclean. It needs some further tweaking. > > Note: I only have partial panorama's based on a lineair lens. I compared 6 > linear lense partial panos. > I have only one 360x180 fish-eye pano set. Yuval "released" a > "hugin_6aroundtilted_testcase" somewhere end of 2007/early 2008. This is my > only but always fisheye 360x180 testcase for hugin builds, including now > with the new CPdetector. > For now I only tested with Pablo's suggested parameters: --grad > --sieve1size 100 --sieve2size 3 -o %o %i > > > Results/Remarks: > - The new CPdetector detects/generates a huge amount of Control Points. > Pano's having approx. 30-40 CPs now have approx. 150-180 CPs. Pano's with > approx. 60-70 CP's now have 220-230 CP's. Yuv's pano now generates 354 CP's > (forgot to note the original number for this mail). After running Celeste > and cpclean generally 25-50% of CP's are removed, which still means that > you have a lot of CP's but the optimizer can easily deal with that. > - The spread of CP's over the overlapping areas is OK, but this might also > be due to the huge amount of CPs. Now is this good, having a good spread of > CP's, or is this bad, having so many CPs? > - Both celeste and cpclean should always be run on the result IMO. Most of > the time I run cpclean 2-3 times (but I also do that when using the > original panomatic and/or autopano-sift-c). > - Speed seems to be comparable with the original panomatic, but I didn't > do stopwatch measurements as I don't care about optimal speed right now. It > is definitely perfectly acceptable :-) > > So far, so good.
[hugin-ptx] Re: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
error LNK2019: unresolved external symbol "__declspec(dllimport) class std::auto_ptr __cdecl vigra::decoder(class vigra::ImageImportInfo const &)" (__imp_?deco...@vigra@@YA?AV? $auto_...@udecoder@vigra@@@std@@abvimageimporti...@1@@Z) referenced in function "void __cdecl vigra::importScalarImage,class vigra::StandardValueAccessor >(class vigra::ImageImportInfo const &,class vigra::BasicImageIterator,class vigra::StandardValueAccessor)" (??$importscalarim...@v? $basicimageitera...@npapan@vigra@@V? $standardvalueacces...@n@2@@vigra@@yaxabvimageimporti...@0@V? $basicimageitera...@npapan@0...@v?$standardvalueaccessor@n...@0@@Z) in keypoints.obj Whatever I try (I've built two versions of the vigra lib, used the downloaded lib, the downloaded dll and the one I built for hugin), I keep getting about 8 errors similar to this. Thomas' tips, that I got offline (thanks!) didn't solve it. Any clues? Allard On Jan 21, 7:07 pm, allard wrote: > @ Harry: I tried those things already before my last post. Both > manually entering the external libs/source and the hugin libs/source. > First I have to hack your Cmake file because it still gives a 'package > not found' error even if I manually enter the lib path (I use cmake > gui 2.8rc5). But neither lib leads to a successful build. Although > after having built the external lib myself the error messages I got > did not seem to have 'vigra' in them. Perhaps it's something else. > > @ Thomas: would you mind sharing the tweaks you did? > > Thanks, Allard > > On Jan 21, 10:16 am, Harry van der Wolf wrote: > > > Hi all, > > > 2010/1/21 T. Modes > > > > I have sucessfull build panomatic-lib with hugins vigraimpex on > > > windows. > > > > Thomas > > > > - > > > @Thomas: > > Thank you for mentioning this. > > Pablo said that we needed an external vigra library but I just tested and it > > does compile against hugins internal library and it also works correctly. > > > @Allard: > > After having built hugin from cmake or MSVC or whatever, you should > > somewhere have a library called huginvigraimpex.dll (or so). > > > If you do a "cmake -DVIGRA_INCLUDE_DIR= > > -D-DVIGRA_LIBRARIES= > > .." > > > and if that doesn't work, does the following work: > > "cmake -DVIGRA_INCLUDE_DIR= > > -D-DVIGRA_LIBRARIES= > > .." > > > Does that work? > > > Harry -- 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: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
@ Harry: I tried those things already before my last post. Both manually entering the external libs/source and the hugin libs/source. First I have to hack your Cmake file because it still gives a 'package not found' error even if I manually enter the lib path (I use cmake gui 2.8rc5). But neither lib leads to a successful build. Although after having built the external lib myself the error messages I got did not seem to have 'vigra' in them. Perhaps it's something else. @ Thomas: would you mind sharing the tweaks you did? Thanks, Allard On Jan 21, 10:16 am, Harry van der Wolf wrote: > Hi all, > > 2010/1/21 T. Modes > > > I have sucessfull build panomatic-lib with hugins vigraimpex on > > windows. > > > Thomas > > > - > > @Thomas: > Thank you for mentioning this. > Pablo said that we needed an external vigra library but I just tested and it > does compile against hugins internal library and it also works correctly. > > @Allard: > After having built hugin from cmake or MSVC or whatever, you should > somewhere have a library called huginvigraimpex.dll (or so). > > If you do a "cmake -DVIGRA_INCLUDE_DIR= > -D-DVIGRA_LIBRARIES= > .." > > and if that doesn't work, does the following work: > "cmake -DVIGRA_INCLUDE_DIR= > -D-DVIGRA_LIBRARIES= > .." > > Does that work? > > Harry -- 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
Re: [hugin-ptx] Re: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
Hi all, 2010/1/21 T. Modes > I have sucessfull build panomatic-lib with hugins vigraimpex on > windows. > > Thomas > > - > @Thomas: Thank you for mentioning this. Pablo said that we needed an external vigra library but I just tested and it does compile against hugins internal library and it also works correctly. @Allard: After having built hugin from cmake or MSVC or whatever, you should somewhere have a library called huginvigraimpex.dll (or so). If you do a "cmake -DVIGRA_INCLUDE_DIR= -D-DVIGRA_LIBRARIES= .." and if that doesn't work, does the following work: "cmake -DVIGRA_INCLUDE_DIR= -D-DVIGRA_LIBRARIES= .." Does that work? Harry -- 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: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
> I'm at a loss here. Does this require other files from vigra than a > build of hugin? The cmake patch does not find the vigra package. I've > tried downloading vigra 1.6 and building the library (changing it to a > static lib, no clue if that works the way I did it), but CMake won't > tell me it has found vigra. I tried overriding the package finder and > putting the paths in manually but that doesn't work either, at least I > get tons of error messages, which seem to be related to vigra if I use > the (renamed) huginvigraimpex.lib from my hugin build or unrelated to > vigra if I use the renamed vigraimpex_dll.lib that I built from the I have sucessfull build panomatic-lib with hugins vigraimpex on windows. But there was more manual tweaking of the project files necessary. With an external vigra 1.6 I had not yet sucess. It compiled, but then it did not run correctly (it was not able to read the files.) So there is some more work with the cmake system to do until it works out of the box. 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
Re: [hugin-ptx] Re: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
Hi Allard, As mentioned: I don't know how Windows handles this. My advise here is only on "best guess". To be able to compile Pablo's source you need an external vigra. The vigra inside hugin does not work. You need both the includes and the dll or static vigra library Now for the Win32 part inside my FindVigra cmake macro (see also below): I checked some other macros for windows stuff and copied from them. IF(WIN32) FIND_PATH(VIGRA_INCLUDE_DIR vigra/gaborfilter.hxx PATHS ${VIGRA_ROOT_PATH}/include ${SOURCE_BASE_DIR}/vigra/include ) # for dynamic build, it's vigraimpex.lib and the dll must be copied into hugin's bin folder #SET(VIGRA_LIBRARIES ${SOURCE_BASE_DIR}/vigra/lib/libvigraindex.dll) FIND_LIBRARY( VIGRALIBRARIES NAMES vigraimpex libvigraimpex PATHS ${VIGRA_ROOT_PATH} ${VIGRA_ROOT_PATH}/Release ${VIGRA_ROOT_PATH}/lib ) ELSE(WIN32) There is a FIND part and a SET part. The Find part really tries to find the path to the includes and to the libraries. You can expand this with the paths where your really have the includes and library (check inside hugin/CMkeModules for examples). The NAMES specifies how the library could be named (without extension, platform independable). If it is called differently on windows please change accordingly. The SET command (disabled here) is not searching but forcing where the library resides. It is up to you if it really is there. You can do the same for the include_dir. Last but not least, you can override it and specify it on the command line: "cmake -DVIGRA_INCLUDE_DIR= -DVIGRA_LIBRARIES= .." Thereby overriding where cmake has to search. I included that in patch version 3 (see attached in the top). IF (VIGRA_INCLUDE_DIR AND VIGRA_LIBRARIES) # in cache already SET(VIGRA_FOUND TRUE) ELSE (VIGRA_INCLUDE_DIR AND VIGRA_LIBRARIES) I hope this helps, but for windows I'm also guessing. Harry 2010/1/21 allard > Harry, Pablo & others, > > I'm at a loss here. Does this require other files from vigra than a > build of hugin? The cmake patch does not find the vigra package. I've > tried downloading vigra 1.6 and building the library (changing it to a > static lib, no clue if that works the way I did it), but CMake won't > tell me it has found vigra. I tried overriding the package finder and > putting the paths in manually but that doesn't work either, at least I > get tons of error messages, which seem to be related to vigra if I use > the (renamed) huginvigraimpex.lib from my hugin build or unrelated to > vigra if I use the renamed vigraimpex_dll.lib that I built from the > downloaded source. > > As usual I'm just touching around in the dark, could somebody give me > a hint? > > Allard > > On Jan 15, 5:31 am, Harry van der Wolf wrote: > > Hi guys, > > > > I'm sorry. I went much too fast. I made the Find module as a lunch > > "practice" and that part works, but I completely forgot to modify the > > CMakeLists.txt files to use the results from my module. > > Please find attached the "version 2" patch. > > > > Harry > > > > 2010/1/15 Harry van der Wolf > > > > > Hi all, > > > > > 2010/1/15 allard > > > > > I can't get it to work yet. > > >> A few steps further already, the changes Pablo made worked out, except > > >> that the "ifdef MSVC" around the USE_MATH_DEFINES was not picked up > > >> somehow, I had to comment it. > > >> I also had to download pthread and use NODEFAULTLIB on msvcprt.lib, > > >> LIBCMT.lib and libcmpt.lib to avoid conflicts with vigraimpex and each > > >> other. > > >> But now I get a load of unresolved external symbol errors. I think the > > >> last two things have something to do with the vigraimpex.lib. I tried > > >> the downloaded release and the one from my hugin builds but neither > > >> would work. > > > > > I created a patch that searches for vigra. It means that you no longer > need > > > to use CFLAGS/CXXFLAGS. > > > > > I patched the CMakeLists.txt and added a FindVIGRA.cmake module in > > > CMakeModules. > > > It works for OSX and should work on all linuxes (but untested). As I > know > > > absolutely nothing of windows it is only an "educated guess" for that > > > platform. > > > I hope Pablo improves the win32 stuff of the FindVIGRA.cmake module. > > > > > Inside panomatic-lib do "patch -p0 < findvigra.diff" > > > > > Harry > > > > > > > > findvigra_v2.diff > > 4KViewDownload > > -- > 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 > -- You received this message because you are subscribed to the Google Groups "hugin and other free panoramic software" group. A list of frequently asked q
[hugin-ptx] Re: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
Harry, Pablo & others, I'm at a loss here. Does this require other files from vigra than a build of hugin? The cmake patch does not find the vigra package. I've tried downloading vigra 1.6 and building the library (changing it to a static lib, no clue if that works the way I did it), but CMake won't tell me it has found vigra. I tried overriding the package finder and putting the paths in manually but that doesn't work either, at least I get tons of error messages, which seem to be related to vigra if I use the (renamed) huginvigraimpex.lib from my hugin build or unrelated to vigra if I use the renamed vigraimpex_dll.lib that I built from the downloaded source. As usual I'm just touching around in the dark, could somebody give me a hint? Allard On Jan 15, 5:31 am, Harry van der Wolf wrote: > Hi guys, > > I'm sorry. I went much too fast. I made the Find module as a lunch > "practice" and that part works, but I completely forgot to modify the > CMakeLists.txt files to use the results from my module. > Please find attached the "version 2" patch. > > Harry > > 2010/1/15 Harry van der Wolf > > > Hi all, > > > 2010/1/15 allard > > > I can't get it to work yet. > >> A few steps further already, the changes Pablo made worked out, except > >> that the "ifdef MSVC" around the USE_MATH_DEFINES was not picked up > >> somehow, I had to comment it. > >> I also had to download pthread and use NODEFAULTLIB on msvcprt.lib, > >> LIBCMT.lib and libcmpt.lib to avoid conflicts with vigraimpex and each > >> other. > >> But now I get a load of unresolved external symbol errors. I think the > >> last two things have something to do with the vigraimpex.lib. I tried > >> the downloaded release and the one from my hugin builds but neither > >> would work. > > > I created a patch that searches for vigra. It means that you no longer need > > to use CFLAGS/CXXFLAGS. > > > I patched the CMakeLists.txt and added a FindVIGRA.cmake module in > > CMakeModules. > > It works for OSX and should work on all linuxes (but untested). As I know > > absolutely nothing of windows it is only an "educated guess" for that > > platform. > > I hope Pablo improves the win32 stuff of the FindVIGRA.cmake module. > > > Inside panomatic-lib do "patch -p0 < findvigra.diff" > > > Harry > > > > findvigra_v2.diff > 4KViewDownload -- 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
Re: [hugin-ptx] Re: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
Hi guys, I'm sorry. I went much too fast. I made the Find module as a lunch "practice" and that part works, but I completely forgot to modify the CMakeLists.txt files to use the results from my module. Please find attached the "version 2" patch. Harry 2010/1/15 Harry van der Wolf > Hi all, > > > 2010/1/15 allard > > I can't get it to work yet. >> A few steps further already, the changes Pablo made worked out, except >> that the "ifdef MSVC" around the USE_MATH_DEFINES was not picked up >> somehow, I had to comment it. >> I also had to download pthread and use NODEFAULTLIB on msvcprt.lib, >> LIBCMT.lib and libcmpt.lib to avoid conflicts with vigraimpex and each >> other. >> But now I get a load of unresolved external symbol errors. I think the >> last two things have something to do with the vigraimpex.lib. I tried >> the downloaded release and the one from my hugin builds but neither >> would work. >> >> > I created a patch that searches for vigra. It means that you no longer need > to use CFLAGS/CXXFLAGS. > > I patched the CMakeLists.txt and added a FindVIGRA.cmake module in > CMakeModules. > It works for OSX and should work on all linuxes (but untested). As I know > absolutely nothing of windows it is only an "educated guess" for that > platform. > I hope Pablo improves the win32 stuff of the FindVIGRA.cmake module. > > Inside panomatic-lib do "patch -p0 < findvigra.diff" > > Harry > > > -- 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 findvigra_v2.diff Description: Binary data
Re: [hugin-ptx] Re: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
Hi all, 2010/1/15 allard > I can't get it to work yet. > A few steps further already, the changes Pablo made worked out, except > that the "ifdef MSVC" around the USE_MATH_DEFINES was not picked up > somehow, I had to comment it. > I also had to download pthread and use NODEFAULTLIB on msvcprt.lib, > LIBCMT.lib and libcmpt.lib to avoid conflicts with vigraimpex and each > other. > But now I get a load of unresolved external symbol errors. I think the > last two things have something to do with the vigraimpex.lib. I tried > the downloaded release and the one from my hugin builds but neither > would work. > > I created a patch that searches for vigra. It means that you no longer need to use CFLAGS/CXXFLAGS. I patched the CMakeLists.txt and added a FindVIGRA.cmake module in CMakeModules. It works for OSX and should work on all linuxes (but untested). As I know absolutely nothing of windows it is only an "educated guess" for that platform. I hope Pablo improves the win32 stuff of the FindVIGRA.cmake module. Inside panomatic-lib do "patch -p0 < findvigra.diff" Harry -- 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 findvigra.diff Description: Binary data
[hugin-ptx] Re: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
I can't get it to work yet. A few steps further already, the changes Pablo made worked out, except that the "ifdef MSVC" around the USE_MATH_DEFINES was not picked up somehow, I had to comment it. I also had to download pthread and use NODEFAULTLIB on msvcprt.lib, LIBCMT.lib and libcmpt.lib to avoid conflicts with vigraimpex and each other. But now I get a load of unresolved external symbol errors. I think the last two things have something to do with the vigraimpex.lib. I tried the downloaded release and the one from my hugin builds but neither would work. Any suggestions? On Jan 14, 3:47 pm, Tduell wrote: > Hullo Harry, > > On Jan 15, 6:24 am, Harry van der Wolf wrote: > [snip]> - cd inside the panomatic-lib source directory (not src) > > - make a build directory like "mkdir build" > > - cd build; > > - issue the following commands to set CFLAGS and so on correctly: > > export PKG_CONFIG_PATH=/opt/local/lib/pkgconfig > > export CFLAGS="-I/opt/local/include -L/opt/local/lib" > > export CXXFLAGS=$CFLAGS > > [snip] > > Thanks for that advice, I used a suitably edited version to build on > Fedora 12. > I have done a couple of tests and it seems to do a good job thus far. > > Cheers, > Terry -- 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: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
Hullo Harry, On Jan 15, 6:24 am, Harry van der Wolf wrote: [snip] > - cd inside the panomatic-lib source directory (not src) > - make a build directory like "mkdir build" > - cd build; > - issue the following commands to set CFLAGS and so on correctly: > export PKG_CONFIG_PATH=/opt/local/lib/pkgconfig > export CFLAGS="-I/opt/local/include -L/opt/local/lib" > export CXXFLAGS=$CFLAGS [snip] Thanks for that advice, I used a suitably edited version to build on Fedora 12. I have done a couple of tests and it seems to do a good job thus far. Cheers, Terry -- 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
Re: [hugin-ptx] Re: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
Hi Skip, 2010/1/14 skip > > > On Jan 13, 6:55 am, Harry van der Wolf wrote: > > > I need more help. I installed vigra and bzr, but I don't know cmake. > How do I tell cmake that the vigraimpex library is in /opt/local/lib ? > > Thanks, > cmake supports builds outside the "root" of the source tree (I forgot the name for it). - cd inside the panomatic-lib source directory (not src) - make a build directory like "mkdir build" - cd build; - issue the following commands to set CFLAGS and so on correctly: export PKG_CONFIG_PATH=/opt/local/lib/pkgconfig export CFLAGS="-I/opt/local/include -L/opt/local/lib" export CXXFLAGS=$CFLAGS - still inside the build directory issue the command "cmake -DCMAKE_BUILD_TYPE=RelWithDebInfo .." or simply "cmake .." - make - (sudo) make install does not work yet, so simply copy from the build directory the panomatic executable to the appropriate place for the bundle (remind that you might have the original panomatic there, so you might copy that out of the way or rename the new panomatic or so) issue the command from build: "cp panomatic/panomatic ~/Library/Application\ Support/Hugin/Autopano/patfreepanomatic" (I renamed it for the time being). Inside your Hugin bundle in preferences for CPDetectors select patfreepanomatic (if you renamed it) and set the parameters to: "--grad --sieve1size 100 --sieve2size 1 -o %o %i" or "--grad --sieve1size 100 --sieve2size 3 -o %o %i". Note: do not delete your build directory yet. Inside "build/localfeatures/" resides "liblocalfeatures.dylib" which is needed by the new panomatic. I did not spend time yet to modify settings like install_name_path. Harry -- 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: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
On Jan 13, 6:55 am, Harry van der Wolf wrote: > All, > > After Pablo's announcement[0] (3 January) of the almost patent free CP > detector I built it on OSX, but due to other priorities I had set myself, I > did not have the time to do tests (sorry Pablo [1]). > > This is only a mail to share some first test results. I need to do more > tests, so don't expect graphical and statistical test results. > == This mail is also to "invite" other users to test as well == > (but I have to admit that I did not create a portable plugin binary yet to > be delivered with Hugin for OSX). > > (My) General conclusion: Based on this prelimiary results and even as it is > now (but that was Pablo's own comment in his mail[0]), I think it does a > good job and I think it is suitable for use with Hugin but in combination > with celeste and cpclean. It needs some further tweaking. > > Note: I only have partial panorama's based on a lineair lens. I compared 6 > linear lense partial panos. > I have only one 360x180 fish-eye pano set. Yuval "released" a > "hugin_6aroundtilted_testcase" somewhere end of 2007/early 2008. This is my > only but always fisheye 360x180 testcase for hugin builds, including now > with the new CPdetector. > For now I only tested with Pablo's suggested parameters: --grad --sieve1size > 100 --sieve2size 3 -o %o %i > > Results/Remarks: > - The new CPdetector detects/generates a huge amount of Control Points. > Pano's having approx. 30-40 CPs now have approx. 150-180 CPs. Pano's with > approx. 60-70 CP's now have 220-230 CP's. Yuv's pano now generates 354 CP's > (forgot to note the original number for this mail). After running Celeste > and cpclean generally 25-50% of CP's are removed, which still means that you > have a lot of CP's but the optimizer can easily deal with that. > - The spread of CP's over the overlapping areas is OK, but this might also > be due to the huge amount of CPs. Now is this good, having a good spread of > CP's, or is this bad, having so many CPs? > - Both celeste and cpclean should always be run on the result IMO. Most of > the time I run cpclean 2-3 times (but I also do that when using the original > panomatic and/or autopano-sift-c). > - Speed seems to be comparable with the original panomatic, but I didn't do > stopwatch measurements as I don't care about optimal speed right now. It is > definitely perfectly acceptable :-) > > So far, so good. > I'm very interested in test results from others. > Apart from the fact that panomatic and autopano-sift-c do a good job, the > license restrictions are by now a real pain. The sooner we have a > patent-free CPdetector the better. > Hoi, > Harry > > [0]:http://groups.google.com/group/hugin-ptx/browse_thread/thread/d9c0558... > [1]:http://www.mail-archive.com/hugin-ptx@googlegroups.com/msg07614.html I need more help. I installed vigra and bzr, but I don't know cmake. How do I tell cmake that the vigraimpex library is in /opt/local/lib ? Thanks, --skip -- 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: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
On Jan 14, 3:57 am, Harry van der Wolf wrote: > Hi Skip, > > 2010/1/14 skip sga...@comcast.net > > > > > Can someone post the right instructions for downloading the branch? I > > installed bazaar using MacPorts, and the command: > > > baz branch lp:~pablo.dangelo/hugin/panomatic-lib > > > results in an error: branch: could not determine source revision from > > directory: /Users/sgaede/development > > This error occurs if you have a library with the same name but obviously > without the (hidden) version info in it. Or you have a corrupt panomatic-lib > library. > > Also, why do you use baz instead of bzr or is it a typo? I installed bzr via > Macports as well and I don't even have that command. > Note (and maybe only for OSX): You need to install vigra as well. Well, not knowing the difference, I installed bazaar, and the command was baz. Just looked in the devel tree and found bzr, bzr-gtk, and two more. I guess I know what to do next. Thanks for the clarification :) > > Harry -- 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: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
allard wrote: I did a first attempt to build on Wndows XP and that was not successful. Let me describe the steps. -Downloading and installing bazaar from their webpage and getting the trunk trough their GUI went smooth. -I ran CMake (i use 2.8.0 RC5 GUI), which went smoothly except that I had to point it to some libraries directories, but I'm used to that from the hugin builds (boost, wxtiff, jpeg, png and zlib). -I opened the 'panomatic.sln' MSVC++ solution (I'm on the 2008 express edition) and pressed 'build'. Many 'file not found' errors came up, which were related to the fact that the boost directory and the vigra directory were not added to the 'additional include directories'. I was a bit surprised about the boost directory as this was certainly in the cmake cache file. Oh, if forgot to add a check for the vigra stuff. The boost include path should be picked up now. -I added the directories through the 'properties/C++' tab of MSVC and rebuilt. This time there were errors in CircularKeyPointDescriptor.cpp , related to M_PI not being defined. Fortunately, some googling told me this is a known problem and that this can be solved by adding '#define _USE_MATH_DEFINES' and '#include ' at the top of the file Fixed in bzr -I rebuilt. This time two more errors came up that I couldn't solve. These were: error C4716: 'PanoDetector::LoadKeypoints' : must return a value' in panodetectorlogic.cp, line 110 This was an error in the source, I just commited a fix. and LINK : fatal error LNK1181: cannot open input file '..\localfeatures \Release\localfeatures.lib' Maybe these are related? Could you update and try again. ciao Pablo -- 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
Re: [hugin-ptx] Re: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
Hi Skip, 2010/1/14 skip sga...@comcast.net > > > Can someone post the right instructions for downloading the branch? I > installed bazaar using MacPorts, and the command: > > baz branch lp:~pablo.dangelo/hugin/panomatic-lib > > results in an error: branch: could not determine source revision from > directory: /Users/sgaede/development > This error occurs if you have a library with the same name but obviously without the (hidden) version info in it. Or you have a corrupt panomatic-lib library. Also, why do you use baz instead of bzr or is it a typo? I installed bzr via Macports as well and I don't even have that command. Note (and maybe only for OSX): You need to install vigra as well. Harry -- 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: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
I did a first attempt to build on Wndows XP and that was not successful. Let me describe the steps. -Downloading and installing bazaar from their webpage and getting the trunk trough their GUI went smooth. -I ran CMake (i use 2.8.0 RC5 GUI), which went smoothly except that I had to point it to some libraries directories, but I'm used to that from the hugin builds (boost, wxtiff, jpeg, png and zlib). -I opened the 'panomatic.sln' MSVC++ solution (I'm on the 2008 express edition) and pressed 'build'. Many 'file not found' errors came up, which were related to the fact that the boost directory and the vigra directory were not added to the 'additional include directories'. I was a bit surprised about the boost directory as this was certainly in the cmake cache file. -I added the directories through the 'properties/C++' tab of MSVC and rebuilt. This time there were errors in CircularKeyPointDescriptor.cpp , related to M_PI not being defined. Fortunately, some googling told me this is a known problem and that this can be solved by adding '#define _USE_MATH_DEFINES' and '#include ' at the top of the file -I rebuilt. This time two more errors came up that I couldn't solve. These were: error C4716: 'PanoDetector::LoadKeypoints' : must return a value' in panodetectorlogic.cp, line 110 and LINK : fatal error LNK1181: cannot open input file '..\localfeatures \Release\localfeatures.lib' Could that last one have something to do with the fact that on windows, the output of 'localfeatures' is a .dll, not a .lib? Just guessing. Anyway, thank you Pablo for the effort, I'd really like to see this work. allard On Jan 13, 3:02 pm, skip wrote: > On Jan 13, 6:55 am, Harry van der Wolf wrote: > > > > > All, > > > After Pablo's announcement[0] (3 January) of the almost patent free CP > > detector I built it on OSX, but due to other priorities I had set myself, I > > did not have the time to do tests (sorry Pablo [1]). > > > This is only a mail to share some first test results. I need to do more > > == This mail is also to "invite" other users to test as well == > > (but I have to admit that I did not create a portable plugin binary yet to > > be delivered with Hugin for OSX). > > > (My) General conclusion: Based on this prelimiary results and even as it is > > now (but that was Pablo's own comment in his mail[0]), I think it does a > > good job and I think it is suitable for use with Hugin but in combination > > with celeste and cpclean. It needs some further tweaking. > > > Note: I only have partial panorama's based on a lineair lens. I compared 6 > > linear lense partial panos. > > I have only one 360x180 fish-eye pano set. Yuval "released" a > > "hugin_6aroundtilted_testcase" somewhere end of 2007/early 2008. This is my > > only but always fisheye 360x180 testcase for hugin builds, including now > > with the new CPdetector. > > For now I only tested with Pablo's suggested parameters: --grad --sieve1size > > 100 --sieve2size 3 -o %o %i > > > Results/Remarks: > > - The new CPdetector detects/generates a huge amount of Control Points. > > Pano's having approx. 30-40 CPs now have approx. 150-180 CPs. Pano's with > > approx. 60-70 CP's now have 220-230 CP's. Yuv's pano now generates 354 CP's > > (forgot to note the original number for this mail). After running Celeste > > and cpclean generally 25-50% of CP's are removed, which still means that you > > have a lot of CP's but the optimizer can easily deal with that. > > - The spread of CP's over the overlapping areas is OK, but this might also > > be due to the huge amount of CPs. Now is this good, having a good spread of > > CP's, or is this bad, having so many CPs? > > - Both celeste and cpclean should always be run on the result IMO. Most of > > the time I run cpclean 2-3 times (but I also do that when using the original > > panomatic and/or autopano-sift-c). > > - Speed seems to be comparable with the original panomatic, but I didn't do > > stopwatch measurements as I don't care about optimal speed right now. It is > > definitely perfectly acceptable :-) > > > So far, so good. > > I'm very interested in test results from others. > > Apart from the fact that panomatic and autopano-sift-c do a good job, the > > license restrictions are by now a real pain. The sooner we have a > > patent-free CPdetector the better. > > Hoi, > > Harry > > > [0]:http://groups.google.com/group/hugin-ptx/browse_thread/thread/d9c0558... > > [1]:http://www.mail-archive.com/hugin-ptx@googlegroups.com/msg07614.html > > Can someone post the right instructions for downloading the branch? I > installed bazaar using MacPorts, and the command: > > baz branch lp:~pablo.dangelo/hugin/panomatic-lib > > results in an error: branch: could not determine source revision from > directory: /Users/sgaede/development > > Thanks, > --skip -- 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 availab
[hugin-ptx] Re: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
On Jan 13, 6:55 am, Harry van der Wolf wrote: > All, > > After Pablo's announcement[0] (3 January) of the almost patent free CP > detector I built it on OSX, but due to other priorities I had set myself, I > did not have the time to do tests (sorry Pablo [1]). > > This is only a mail to share some first test results. I need to do more > tests, so don't expect graphical and statistical test results. > == This mail is also to "invite" other users to test as well == > (but I have to admit that I did not create a portable plugin binary yet to > be delivered with Hugin for OSX). > > (My) General conclusion: Based on this prelimiary results and even as it is > now (but that was Pablo's own comment in his mail[0]), I think it does a > good job and I think it is suitable for use with Hugin but in combination > with celeste and cpclean. It needs some further tweaking. > > Note: I only have partial panorama's based on a lineair lens. I compared 6 > linear lense partial panos. > I have only one 360x180 fish-eye pano set. Yuval "released" a > "hugin_6aroundtilted_testcase" somewhere end of 2007/early 2008. This is my > only but always fisheye 360x180 testcase for hugin builds, including now > with the new CPdetector. > For now I only tested with Pablo's suggested parameters: --grad --sieve1size > 100 --sieve2size 3 -o %o %i > > Results/Remarks: > - The new CPdetector detects/generates a huge amount of Control Points. > Pano's having approx. 30-40 CPs now have approx. 150-180 CPs. Pano's with > approx. 60-70 CP's now have 220-230 CP's. Yuv's pano now generates 354 CP's > (forgot to note the original number for this mail). After running Celeste > and cpclean generally 25-50% of CP's are removed, which still means that you > have a lot of CP's but the optimizer can easily deal with that. > - The spread of CP's over the overlapping areas is OK, but this might also > be due to the huge amount of CPs. Now is this good, having a good spread of > CP's, or is this bad, having so many CPs? > - Both celeste and cpclean should always be run on the result IMO. Most of > the time I run cpclean 2-3 times (but I also do that when using the original > panomatic and/or autopano-sift-c). > - Speed seems to be comparable with the original panomatic, but I didn't do > stopwatch measurements as I don't care about optimal speed right now. It is > definitely perfectly acceptable :-) > > So far, so good. > I'm very interested in test results from others. > Apart from the fact that panomatic and autopano-sift-c do a good job, the > license restrictions are by now a real pain. The sooner we have a > patent-free CPdetector the better. > Hoi, > Harry > > [0]:http://groups.google.com/group/hugin-ptx/browse_thread/thread/d9c0558... > [1]:http://www.mail-archive.com/hugin-ptx@googlegroups.com/msg07614.html Can someone post the right instructions for downloading the branch? I installed bazaar using MacPorts, and the command: baz branch lp:~pablo.dangelo/hugin/panomatic-lib results in an error: branch: could not determine source revision from directory: /Users/sgaede/development Thanks, --skip -- 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
Re: [hugin-ptx] Re: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
It would be great to see it. Nathan allard wrote: Did anyone attempt a build for Windows yet? I'll give it a go. Allard On Jan 13, 4:38 am, "bruno.postle" wrote: On Jan 13, 11:55 am, Harry van der Wolf wrote: After Pablo's announcement[0] (3 January) of the almost patent free CP detector I built it on OSX, but due to other priorities I had set myself, I did not have the time to do tests (sorry Pablo [1]). Just adding that it builds on fedora F12 with no problems, but I haven't had a chance to run it yet. -- 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: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
Did anyone attempt a build for Windows yet? I'll give it a go. Allard On Jan 13, 4:38 am, "bruno.postle" wrote: > On Jan 13, 11:55 am, Harry van der Wolf wrote: > > > > > After Pablo's announcement[0] (3 January) of the almost patent free CP > > detector I built it on OSX, but due to other priorities I had set myself, I > > did not have the time to do tests (sorry Pablo [1]). > > Just adding that it builds on fedora F12 with no problems, but I > haven't had a chance to run it yet. > > -- > 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: Tests for Pablo's (almost) patent free CP detector (was: Towards a non-patented control point detector)
On Jan 13, 11:55 am, Harry van der Wolf wrote: > > After Pablo's announcement[0] (3 January) of the almost patent free CP > detector I built it on OSX, but due to other priorities I had set myself, I > did not have the time to do tests (sorry Pablo [1]). Just adding that it builds on fedora F12 with no problems, but I haven't had a chance to run it yet. -- 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