From: Daniel M. G. <dmg...@uv...> - 2006-01-16 00:48:38
|
A new version of PTmender is already in CVS. The changes are mainly support for Photoshop PSD format, including the creation of layer masks. There are two caveats, however: I have not implemented 16bit support in masks yet, and there is no feathering support either. In my opinion, PTmender is ready for real use. I will probably take a break from development for a while. I need to get back to take photos, after all :) My plans for PTmender are: In the short while, and in this order: * Add support for flat output, including JPG (1 single layer) * Add support for 16 bit creation of masks. * Add feathering support. In the longer term: * Add support for 16 bit colour-brightness correction. Also, PTmender hasn't been tested in correction of intensity only, nor in correction of colours only (b and d options). It has only been tested in the k option. Use at your own risk. And there are some parts of PTmender that I don't care to implement: * Support for VRML, PICT, QTVR, IVR * Support for other-than-rectilinear lenses -- Daniel M. German "As De Gaulle used to say: 'Aim well, shoot fast Henri Cartier Bresson -> and get the hell out.'" http://turingmachine.org/ http://silvernegative.com/ dmg (at) uvic (dot) ca replace (at) with @ and (dot) with . |
From: Bruno P. <br...@po...> - 2006-01-24 12:03:52
|
Hi, I thought I'd try to build some windows binaries of ptmender and put them online. Compilation works fine for me on linux, but with MinGW I get this error: gcc -g -O2 -o .libs/PTmender.exe PTmender.o ColourBrightness.o -L/home/Bruno.Postle/libpano /home/Bruno.Postle/libpano/.libs/libpano12.dll.a -L/usr/local/lib -lpng -ljpeg /usr/local/lib/libtiff.dll.a -lz -L/usr/local/lib PTmender.o(.text+0x310b): In function `main': C:/msys/1.0/home/Bruno.Postle/libpano/tools/PTmender.c:297: undefined reference to `InsertFileName' PTmender.o(.text+0x3285):C:/msys/1.0/home/Bruno.Postle/libpano/tools/PTmender.c:365: undefined reference to `InsertFileName' To get it working, I pasted this function from ptpicker.c into PTmender.c: void InsertFileName( fullPath *fp, char *fname ){ char *c = strrchr((char*)(fp->name), PATH_SEP); if(c != NULL) c++; else c = fp->name; strcpy( c, fname ); } I'm building using my own instructions from the wiki: http://www.panotools.info/mediawiki/index.php?title=Build_pano12_from_sourcecode BTW, This page needs fixing as all the binaries end-up dynamically linked to libtiff and therefore not much use. I don't know enough about this platform to figure-out the problem - Somebody else needs to step in here. -- Bruno |
From: Daniel M. G. <dmg...@uv...> - 2006-01-24 15:46:52
|
Bruno Postle twisted the bytes to say: Bruno> To get it working, I pasted this function from ptpicker.c into Bruno> PTmender.c: Bruno> void InsertFileName( fullPath *fp, char *fname ){ char *c = strrchr((char*)(fp->name), PATH_SEP); if(c != NULL) c++; else c = fp->name; strcpy( c, fname ); Bruno> } Hi Bruno, this explains why InsertFileName is in the binary of PTstitcher, even though it is part of the library (at least under Linux). I guess that in some architectures it gets built into the library and in some it is not. Search for InsertFileName in PTmender, you will see that the code is '#ifdef'-ed out. I think that the way to fix this is make InsertFileName available within the library. Anybody with Windows experience to help us here? Bruno> I'm building using my own instructions from the wiki: Bruno> http://www.panotools.info/mediawiki/index.php?title=Build_pano12_from_sourcecode Bruno> BTW, This page needs fixing as all the binaries end-up Bruno> dynamically linked to libtiff and therefore not much use. I Bruno> don't know enough about this platform to figure-out the Bruno> problem - Somebody else needs to step in here. -- Daniel M. German "Any sufficiently advanced technology is indistinguishable Arthur C. Clarke -> from magic." http://turingmachine.org/ http://silvernegative.com/ dmg (at) uvic (dot) ca replace (at) with @ and (dot) with . |
From: Bruno P. <br...@po...> - 2006-02-07 09:59:51
|
On Tue 24-Jan-2006 at 11:55 +0000, Bruno Postle wrote: > > I'm building using my own instructions from the wiki: > > http://www.panotools.info/mediawiki/index.php?title=Build_pano12_from_sourcecode This page is fixed, these instructions now result in working win32 binaries (an older version of libtiff is required to get it to work). I put a zip file with a statically linked pano12.dll, PTmender.exe and PTblender.exe here: http://bugbear.blackfish.org.uk/~bruno/pano12-2.8.0-with-tools.zip Thanks to Erik Kraus who discovered the reason why I couldn't get either working, the command line is space-sensitive on windows so these crash: PTblender.exe -o blended -k 2 foo000*.tif PTmender.exe -o mended script.txt ...but these work fine: PTblender.exe -oblended -k2 foo000*.tif PTmender.exe -omended script.txt -- Bruno |
From: Daniel M. G. <dmg...@uv...> - 2006-02-07 11:51:41
|
Bruno Postle twisted the bytes to say: Bruno> On Tue 24-Jan-2006 at 11:55 +0000, Bruno Postle wrote: >> >> I'm building using my own instructions from the wiki: >> >> http://www.panotools.info/mediawiki/index.php?title=Build_pano12_from_sourcecode Bruno> This page is fixed, these instructions now result in working win32 Bruno> binaries (an older version of libtiff is required to get it to Bruno> work). I put a zip file with a statically linked pano12.dll, Bruno> PTmender.exe and PTblender.exe here: Mmm, is this something that only occurs under windows? Bruno> http://bugbear.blackfish.org.uk/~bruno/pano12-2.8.0-with-tools.zip Bruno> Thanks to Erik Kraus who discovered the reason why I couldn't get Bruno> either working, the command line is space-sensitive on windows so Bruno> these crash: PTblender.exe -o blended -k 2 foo000*.tif PTmender.exe -o mended script.txt Bruno> ...but these work fine: PTblender.exe -oblended -k2 foo000*.tif PTmender.exe -omended script.txt Bruno> -- Bruno> Bruno Wow, talk about platform dependencies. I'll update the documentation accordingly. I'll also make the help be OS sensitive. I have also finished a program that merges TIFFs (8 bits only) into a PSD. I'll try to upload it by the end of the week. dmg Bruno> ------------------------------------------------------- Bruno> This SF.net email is sponsored by: Splunk Inc. Do you grep through log files Bruno> for problems? Stop! Download the new AJAX search engine that makes Bruno> searching your log files as easy as surfing the web. DOWNLOAD SPLUNK! Bruno> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642 Bruno> _______________________________________________ Bruno> PanoTools-devel mailing list Bruno> Pan...@li... Bruno> https://lists.sourceforge.net/lists/listinfo/panotools-devel -- Daniel M. German "As De Gaulle used to say: 'Aim well, shoot fast Henri Cartier Bresson -> and get the hell out.'" http://turingmachine.org/ http://silvernegative.com/ dmg (at) uvic (dot) ca replace (at) with @ and (dot) with . |
From: Bruno P. <br...@po...> - 2006-02-07 12:24:00
|
On Tue 07-Feb-2006 at 12:51 +0100, Daniel M. German wrote: > > > > http://www.panotools.info/mediawiki/index.php?title=Build_pano12_from_sourcecode > > > This page is fixed, these instructions now result in working > > win32 binaries (an older version of libtiff is required to get > > it to work). > Mmm, is this something that only occurs under windows? Probably, I have built pano12 on linux *lots* of times over the last three years - There has never been a problem with any version of libtiff. -- Bruno |
From: <pab...@in...> - 2006-02-07 18:49:15
|
Hi Daniel, > Bruno> Thanks to Erik Kraus who discovered the reason why I couldn't get > Bruno> either working, the command line is space-sensitive on windows so > Bruno> these crash: > > PTblender.exe -o blended -k 2 foo000*.tif > PTmender.exe -o mended script.txt > > Bruno> ...but these work fine: > > PTblender.exe -oblended -k2 foo000*.tif > PTmender.exe -omended script.txt > > Bruno> -- > Bruno> Bruno > > Wow, talk about platform dependencies. I'll update the documentation > accordingly. I'll also make the help be OS sensitive. Isn't it possible to have a uniform interface on all platforms? if the problem is with the getopt() function provided by mingw, just replace it with a better one... (The hugin code contains a copy of one that doesn't have this problem.) Little things like that make it harder to use PTmender etc. in cross platform applications like hugin. ciao Pablo ---------------------------------------------------------------- This message was sent using IMP, the Internet Messaging Program. |
From: Daniel M. G. <dmg...@uv...> - 2006-02-07 20:05:09
|
Hi Pablo, pablo> Isn't it possible to have a uniform interface on all platforms? if the problem I agree with you. We should strive for that. pablo> is with the getopt() function provided by mingw, just replace it with a better I am not sure what you mean by mingw. Under OS X getopt is part of the BSD library. I suspect that the getopt under Linux is the GNU one. I guess that what we need is to provide a replacement _only_ for Windows, am I correct? I'll look at the one provided by hugin. daniel P.S. This is the man page of getopt under OS X. ---------------------------------------------------------------------- GETOPT(3) BSD Library Functions Manual GETOPT(3) NAME getopt - get option character from command line argument list LIBRARY Standard C Library (libc, -lc) SYNOPSIS #include <unistd.h> extern char *optarg; extern int optind; extern int optopt; extern int opterr; extern int optreset; int getopt(int argc, char * const *argv, const char *optstring) ---------------------------------------------------------------------- pablo> one... (The hugin code contains a copy of one that doesn't have this problem.) pablo> Little things like that make it harder to use PTmender etc. in cross platform pablo> applications like hugin. Understood. pablo> ciao -- Daniel M http://turingmachine.org/ http://silvernegative.com/ dmg (at) uvic (dot) ca replace (at) with @ and (dot) with . |
From: <jo...@jo...> - 2006-02-07 20:24:38
|
Could this just be a simple syntactical mistake? I emailed the list 2006-01-23 the following: > There was a minor problem for me under Win32, > > while ((opt = getopt(argc, argv, "o::k:hqc")) != -1) { > didn't work, so I just got rid of the extra colon > > while ((opt = getopt(argc, argv, "o:k:hqc")) != -1) { > and it then worked fine. I think the problem is that double colon. I looked in the getopt specs, and could find no meaning for having two together; one colon means that option is optional. In the CVS I still see the double colon present for PTblender.c, but not PTmender.c. Would be good to see if changing this fixes your problems. -josh >> Bruno> Thanks to Erik Kraus who discovered the reason why I couldn't >> get >> Bruno> either working, the command line is space-sensitive on windows >> so >> Bruno> these crash: >> >> PTblender.exe -o blended -k 2 foo000*.tif >> PTmender.exe -o mended script.txt >> >> Bruno> ...but these work fine: >> >> PTblender.exe -oblended -k2 foo000*.tif >> PTmender.exe -omended script.txt >> >> Bruno> -- >> Bruno> Bruno |