Bug 286633

Summary:	Unable to find cpfind executable, but cpfind is installed and availabe within the path
Product:	[Applications] digikam	Reporter:	tps
Component:	Plugin-Generic-Panorama	Assignee:	Digikam Developers <digikam-bugs-null>
Status:	RESOLVED FIXED
Severity:	crash	CC:	benjamin.girault, caulier.gilles, philip.johnsson
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:		Version Fixed In:	2.6.0
Sentry Crash Report:
Attachments:	hugin project file hugin make file cp_pano.pto cp_pano_clean.pto pano_base.pto

Description tps 2011-11-14 22:21:05 UTC

Version:           2.3.0 (using KDE 4.7.3) 
OS:                Linux

panoramagui doesn't find cpfind, but cpfind is available and within the search path:

$ which cpfind
/usr/bin/cpfind
$ echo $PATH
/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games

Looking for very same bugs I've found: https://bugs.kde.org/show_bug.cgi?id=281794 leading to https://bugs.kde.org/show_bug.cgi?id=280736 describing the same behaviour.

Reproducible: Always

Steps to Reproduce:
Install digikam from philips5-ppa, install hugin from hugin-nightly-ppa (you may use the one from philips5-ppa, but this one gives an other error). Then call "panoramagui" directly or via digikam.

Actual Results:  
Tells missing cpfind, but cpfind is there and can be called:
$ cpfind --version
Hugin's cpfind 2011.4.0.9d834b027bef
based on Pan-o-matic by Anael Orlinski
my version message: 0.1

Expected Results:  
panoramagui running creating a panorama from given files

Comment 1 Benjamin Girault 2011-11-14 22:35:06 UTC

Thanks for the bug report, I'll patch the repository ASAP.

Comment 2 Benjamin Girault 2011-11-14 23:03:16 UTC

Git commit e50d59c98362688609107de0ca4f654880cc68c5 by Benjamin Girault.
Committed on 15/11/2011 at 00:06.
Pushed by girault into branch 'master'.

Panorama plugin: bugfix cpfind detection

Solves an issue with cpfind version headline with Hugin 2011.4.
CCBUG: 286633

M  +11   -0    panorama/manager/cpfindbinary.cpp

http://commits.kde.org/kipi-plugins/e50d59c98362688609107de0ca4f654880cc68c5

Comment 3 caulier.gilles 2011-11-15 09:33:59 UTC

tps,

The solution from Benjamin included in git/master (next 2.4.0) work for you ?

Thanks in advance

Gilles Caulier

Comment 4 tps 2011-11-15 23:19:18 UTC

Do not have a system set up to compile this for Ubuntu 11.10. Any repository with compiled versions available??

Comment 5 caulier.gilles 2011-11-16 06:09:09 UTC

Not yet. next release including this patch will be 2.4.0, planed at first week of december...

Gilles Caulier

Comment 6 caulier.gilles 2011-12-20 10:21:16 UTC

TPS,

kipi-plugins 2.4 is out. Can you reproduce the problem ?

Gilles Caulier

Comment 7 tps 2011-12-20 20:07:25 UTC

Installed are:
ii  kipi-plugins        2:2.4.1-oneiric~ppa1 image manipulation/handling plugins for KIPI aware programs
ii  kipi-plugins-common 2:2.4.1-oneiric~ppa1 kipi-plugins architecture-independent data
ii  libkipi-data        4:4.7.4-oneiric~ppa1 data files for kipi libraries
ii  libkipi8            4:4.7.4-oneiric~ppa1 library for apps that want to use kipi-plugin

Stitching three images together (I've done this successfully) gives back:

Panorama compilation: Cannot run pto2mk:

 number of cmdline args: 1
WARN: 20:58:32.603318 (/build/buildd/hugin-2011.4.0/src/hugin_base/panodata/Panorama.cpp:1792) readData(): Failed to read from dataInput.
error while parsing panos tool script: /tmp/kde-tps/kipi-panorama-preprocessing-tmp-1324411088zYe8YD/auto_op_pano.pto

Unsure if this is a hugin problem or one related to panorama, because starting hugin and stitching the very same pictures:
Der Assistent hat nicht erfolgreich abgeschlossen. Bitte überprüfen Sie die erzeugte Projektdatei.

or in english:
The assistant did not close successful. Please have a look at the created project file.

hugin is version:
ii  hugin       2011.4.0-oneiric~ppa1 panorama photo stitcher - GUI tools
ii  hugin-data  2011.4.0-oneiric~ppa1 panorama photo stitcher - common data files
ii  hugin-tools 2011.4.0-oneiric~ppa1 panorama photo stitcher - commandline tools

Comment 8 tps 2011-12-20 20:09:06 UTC

Created attachment 66950 [details]
hugin project file

Comment 9 tps 2011-12-20 20:09:43 UTC

Created attachment 66951 [details]
hugin make file

Comment 10 tps 2011-12-20 20:14:28 UTC

Created attachment 66952 [details]
cp_pano.pto

Comment 11 tps 2011-12-20 20:15:02 UTC

Created attachment 66953 [details]
cp_pano_clean.pto

Comment 12 tps 2011-12-20 20:15:30 UTC

Created attachment 66954 [details]
pano_base.pto

Comment 13 tps 2011-12-20 20:19:12 UTC

cp_pano.pto
cp_pano_clean.pto
pano_base.pto

are created by the panorama plugin. They exist within the directory the plugin reports it could not read the files. The file the plugin wants to read, indeed doesn't exist and is never created: "/tmp/kde-tps/kipi-panorama-preprocessing-tmp-1324411979hedhj1/auto_op_pano.pto".

Comment 14 tps 2011-12-20 20:22:51 UTC

At the moment I'd say:
* hugin seems broken (but not the libraries, since other tools depending on them work).
* the panorama plugin is broken too, since it wants to read a file never created.

Comment 15 Philip Johnsson 2011-12-22 16:41:54 UTC

Does this happen if you do the same thing from the true Hugin gui or only with the panorama tool in Digikam? If so it's a Hugin bug I have to look into for my packages.

I just updated Hugin to the at the moment latest version 2011.4.0 on my PPA. I see that you use that one. Hugin have introduced a new vertical feature detection tool. Not sure if that effect the panorama plugin with Digikam.

Latest version have this signature:

$ cpfind --version
Hugin's cpfind 2011.4.0.cf9be9344356
based on Pan-o-matic by Anael Orlinski
my version message: 0.1

/Philip


(In reply to comment #14)
> At the moment I'd say:
> * hugin seems broken (but not the libraries, since other tools depending on
> them work).
> * the panorama plugin is broken too, since it wants to read a file never
> created.

Comment 16 Benjamin Girault 2012-01-01 18:06:52 UTC

tps: Can you add another bug report for this latest issue? It is definitely a different issue than the first one.

Moreover, could you include the full console log of panoramagui (I am interested in why autooptimiser didn't create the missing file) in the new bug report?

Thanks.