Bug 325830 - Panorama Stitcher returns an error after pre-processing, as a result of attempting to invoke Hugin commands
Summary: Panorama Stitcher returns an error after pre-processing, as a result of attem...
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Plugin-Generic-Panorama (show other bugs)
Version: unspecified
Platform: Microsoft Windows Microsoft Windows
: NOR major
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-10-09 22:29 UTC by Jim Rowley
Modified: 2016-07-07 04:15 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 5.0.0


Attachments
attachment-23323-0.html (1.64 KB, text/html)
2015-05-10 16:44 UTC, Jim Rowley
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jim Rowley 2013-10-09 22:29:41 UTC
The issue is Digikam Panorama stitcher fails after pre-processing to execute the Hugin files needed.  I continually get an error message.
I'm using:
Windows 8 64 bit (but does the same with Windows XP sp3)
Digikam 3.4.0 (but failed exactly the same on  3.3.0)
Hugin 12.0.0
 
Everything goes well  through pre-processing, then, I hit  next and it processes for  a very short  time and  comes back with the following:
 
ERROR
 
 
Cannot run C:/Program Files/Hugin/bin/make.exe:
'C:/Program Files/Hugin/bin/nona.exe' -z LZW -r ldr -mTIFF__m -o “preview” -i 0 “C:/Users/Jim/AppData/Roaming/.kde/tmp-ACER “C:' is not recognized as an internal or external command, operable program or batch file.
Make: ***[preview0000.tif]Error 1
 
It seems to me that the directory and file delimiters for windows are wrong and these command lines should look more like  this
 
C:\Program Files\Hugin\bin\make.exe
C:\Program Files\Hugin\bin\nona.exe -z LZW -r ldr -m TIFF__m -o “preview” -i 0 C:\Users\Jim\AppData\Roaming\.kde\tmp-ACER-PC
 
At least as far as the delimiters are concerned.

It looks like a command and parameter string being sent to the Windows command processor to run "make" and "nona" and pass parameters to the program, but the back slashes in the folder\file structure are telling the command processor that "C:" is the program to run and the rest are parameters to pass.
 
The initial start-up of the panorama stitcher acknowledges the Hugin files with a green check on each.  I tried re-browsing each one and also tried dumping the whole Hugin bin contents into Digikam bin file and re-associating them, but neither worked.  Each method showed a green checkmark, but the same  error message.  I uninstalled and  reinstalled Digikam to remove my file transfer, then looked into the windows registry to see the file associations were stored there, but no luck finding anything I could correct in the registry.  I tried to find a readable database the paths may be stored in, but gave up. I'm now at a loss.
 
Hugin creates panoramas just fine, so I don't think it has anything to do with their programs and I cannot find this specific issue on the internet.  

Reproducible: Always

Steps to Reproduce:
1.select files to stitch and launch stitcher from drop-down menu
2.use default options (or not) and select next
3.after pre-processing, select next
Actual Results:  
Error message:
select cancel

Expected Results:  
Produce the panorama

Different options seem to make no difference.
Comment 1 caulier.gilles 2015-05-10 09:06:20 UTC
Problem still reproducible with kipi-plugins 4.9.0 ?

Gilles Caulier
Comment 2 Jim Rowley 2015-05-10 16:44:36 UTC
Created attachment 92527 [details]
attachment-23323-0.html

I don't know. I gave up on digikam a year ago. I may install the latest version and see what happens. I'll report back if the same issue occurs.

      From: Gilles Caulier <caulier.gilles@gmail.com>
 To: jrowley1961@yahoo.com 
 Sent: Sunday, May 10, 2015 2:06 AM
 Subject: [kipiplugins] [Bug 325830] Panorama Stitcher returns an error after pre-processing, as a result of attempting to invoke Hugin commands
   
https://bugs.kde.org/show_bug.cgi?id=325830

--- Comment #1 from Gilles Caulier <caulier.gilles@gmail.com> ---
Problem still reproducible with kipi-plugins 4.9.0 ?

Gilles Caulier
Comment 3 caulier.gilles 2015-05-18 15:27:46 UTC
What's new about this file using last digiKam 4.10.0 ?

Gilles Caulier
Comment 4 caulier.gilles 2015-07-06 07:35:16 UTC
digiKam 4.11.0 Windows installer is available for download :

http://download.kde.org/stable/digikam/digiKam-installer-4.11.0-win32.exe.mirrorlist
Comment 5 caulier.gilles 2015-08-13 08:10:23 UTC
digiKam 4.12.0 is out.

https://www.digikam.org/node/741

Problem still reproducible ?

Gilles Caulier
Comment 6 caulier.gilles 2016-02-21 11:04:37 UTC
Git commit 1a5a6ea49710bed310875900814be70744385aa5 by Gilles Caulier.
Committed on 21/02/2016 at 10:57.
Pushed by cgilles into branch 'master'.

PAnorama tool integration to digiKam core.
Tested with JPEG pano using Hugin 2015.0. Work perfectly.
Nedd to perform regression test with RAW files.
Stand alone version restored into core/test/assistants.
Related: bug 351433
FIXED-IN: 5.0.0

M  +8    -1    CMakeLists.txt
M  +18   -4    app/CMakeLists.txt
M  +16   -0    app/main/digikamapp.cpp
M  +1    -0    app/main/digikamapp.h
M  +2    -0    app/main/digikamapp_p.h
M  +2    -1    app/main/digikamui.rc
M  +1    -0    data/icons/CMakeLists.txt
A  +-    --    data/icons/panorama/128-apps-kipi-panorama.png
A  +-    --    data/icons/panorama/16-apps-kipi-panorama.png
A  +-    --    data/icons/panorama/22-apps-kipi-panorama.png
A  +-    --    data/icons/panorama/32-apps-kipi-panorama.png
A  +-    --    data/icons/panorama/48-apps-kipi-panorama.png
A  +-    --    data/icons/panorama/96-apps-kipi-panorama.png
A  +15   -0    data/icons/panorama/CMakeLists.txt
A  +-    --    data/icons/panorama/sc-apps-kipi-panorama.svgz
M  +1    -0    data/pics/CMakeLists.txt
A  +-    --    data/pics/assistant-hugin.png
M  +44   -9    tests/assistants/CMakeLists.txt
A  +44   -0    tests/assistants/panoparser_test.cpp     [License: GPL (v2+)]
A  +66   -0    tests/assistants/panorama.cpp     [License: GPL (v2+)]
M  +4    -0    utilities/assistants/CMakeLists.txt
M  +1    -1    utilities/assistants/common/dprogresswdg.cpp
A  +77   -0    utilities/assistants/panorama/CMakeLists.txt
A  +34   -0    utilities/assistants/panorama/README
A  +285  -0    utilities/assistants/panorama/importwizard/panointropage.cpp     [License: GPL (v2+)]
A  +74   -0    utilities/assistants/panorama/importwizard/panointropage.h     [License: GPL (v2+)]
A  +122  -0    utilities/assistants/panorama/importwizard/panoitemspage.cpp     [License: GPL (v2+)]
A  +66   -0    utilities/assistants/panorama/importwizard/panoitemspage.h     [License: GPL (v2+)]
A  +332  -0    utilities/assistants/panorama/importwizard/panolastpage.cpp     [License: GPL (v2+)]
A  +74   -0    utilities/assistants/panorama/importwizard/panolastpage.h     [License: GPL (v2+)]
A  +375  -0    utilities/assistants/panorama/importwizard/panooptimizepage.cpp     [License: GPL (v2+)]
A  +73   -0    utilities/assistants/panorama/importwizard/panooptimizepage.h     [License: GPL (v2+)]
A  +369  -0    utilities/assistants/panorama/importwizard/panopreprocesspage.cpp     [License: GPL (v2+)]
A  +72   -0    utilities/assistants/panorama/importwizard/panopreprocesspage.h     [License: GPL (v2+)]
A  +632  -0    utilities/assistants/panorama/importwizard/panopreviewpage.cpp     [License: GPL (v2+)]
A  +79   -0    utilities/assistants/panorama/importwizard/panopreviewpage.h     [License: GPL (v2+)]
A  +131  -0    utilities/assistants/panorama/importwizard/panowizard.cpp     [License: GPL (v2+)]
A  +54   -0    utilities/assistants/panorama/importwizard/panowizard.h     [License: GPL (v2+)]
A  +60   -0    utilities/assistants/panorama/manager/autooptimiserbinary.h     [License: GPL (v2+)]
A  +57   -0    utilities/assistants/panorama/manager/cpcleanbinary.h     [License: GPL (v2+)]
A  +60   -0    utilities/assistants/panorama/manager/cpfindbinary.cpp     [License: GPL (v2+)]
A  +70   -0    utilities/assistants/panorama/manager/cpfindbinary.h     [License: GPL (v2+)]
A  +61   -0    utilities/assistants/panorama/manager/enblendbinary.h     [License: GPL (v2+)]
A  +58   -0    utilities/assistants/panorama/manager/huginexecutorbinary.h     [License: GPL (v2+)]
A  +61   -0    utilities/assistants/panorama/manager/makebinary.h     [License: GPL (v2+)]
A  +61   -0    utilities/assistants/panorama/manager/nonabinary.h     [License: GPL (v2+)]
A  +118  -0    utilities/assistants/panorama/manager/panoactions.h     [License: GPL (v2+)]
A  +510  -0    utilities/assistants/panorama/manager/panoactionthread.cpp     [License: GPL (v2+)]
A  +118  -0    utilities/assistants/panorama/manager/panoactionthread.h     [License: GPL (v2+)]
A  +607  -0    utilities/assistants/panorama/manager/panomanager.cpp     [License: GPL (v2+)]
A  +140  -0    utilities/assistants/panorama/manager/panomanager.h     [License: GPL (v2+)]
A  +57   -0    utilities/assistants/panorama/manager/panomodifybinary.h     [License: GPL (v2+)]
A  +61   -0    utilities/assistants/panorama/manager/pto2mkbinary.h     [License: GPL (v2+)]
A  +1100 -0    utilities/assistants/panorama/ptoparser/panoParser.y     [License: GPL (v2+)]
A  +333  -0    utilities/assistants/panorama/ptoparser/panoScanner.l     [License: GPL (v2+)]
A  +438  -0    utilities/assistants/panorama/ptoparser/ptofile.cpp     [License: GPL (v2+)]
A  +59   -0    utilities/assistants/panorama/ptoparser/ptofile.h     [License: GPL (v2+)]
A  +229  -0    utilities/assistants/panorama/ptoparser/tparser.c     [License: GPL (v2+)]
A  +232  -0    utilities/assistants/panorama/ptoparser/tparser.h     [License: GPL (v2+)]
A  +42   -0    utilities/assistants/panorama/ptoparser/tparserdebug.h     [License: GPL (v2+)]
A  +965  -0    utilities/assistants/panorama/ptoparser/tparsergetters.c     [License: GPL (v2+)]
A  +166  -0    utilities/assistants/panorama/ptoparser/tparsergetters.h     [License: GPL (v2+)]
A  +244  -0    utilities/assistants/panorama/ptoparser/tparserprivate.c     [License: GPL (v2+)]
A  +49   -0    utilities/assistants/panorama/ptoparser/tparserprivate.h     [License: GPL (v2+)]
A  +401  -0    utilities/assistants/panorama/ptotype/ptotype.cpp     [License: GPL (v2+)]
A  +271  -0    utilities/assistants/panorama/ptotype/ptotype.h     [License: GPL (v2+)]
A  +72   -0    utilities/assistants/panorama/tasks/autocroptask.cpp     [License: GPL (v2+)]
A  +56   -0    utilities/assistants/panorama/tasks/autocroptask.h     [License: GPL (v2+)]
A  +107  -0    utilities/assistants/panorama/tasks/commandtask.cpp     [License: GPL (v2+)]
A  +66   -0    utilities/assistants/panorama/tasks/commandtask.h     [License: GPL (v2+)]
A  +70   -0    utilities/assistants/panorama/tasks/compilemksteptask.cpp     [License: GPL (v2+)]
A  +59   -0    utilities/assistants/panorama/tasks/compilemksteptask.h     [License: GPL (v2+)]
A  +63   -0    utilities/assistants/panorama/tasks/compilemktask.cpp     [License: GPL (v2+)]
A  +61   -0    utilities/assistants/panorama/tasks/compilemktask.h     [License: GPL (v2+)]
A  +220  -0    utilities/assistants/panorama/tasks/copyfilestask.cpp     [License: GPL (v2+)]
A  +64   -0    utilities/assistants/panorama/tasks/copyfilestask.h     [License: GPL (v2+)]
A  +66   -0    utilities/assistants/panorama/tasks/cpcleantask.cpp     [License: GPL (v2+)]
A  +54   -0    utilities/assistants/panorama/tasks/cpcleantask.h     [License: GPL (v2+)]
A  +70   -0    utilities/assistants/panorama/tasks/cpfindtask.cpp     [License: GPL (v2+)]
A  +54   -0    utilities/assistants/panorama/tasks/cpfindtask.h     [License: GPL (v2+)]
A  +75   -0    utilities/assistants/panorama/tasks/createfinalptotask.cpp     [License: GPL (v2+)]
A  +54   -0    utilities/assistants/panorama/tasks/createfinalptotask.h     [License: GPL (v2+)]
A  +83   -0    utilities/assistants/panorama/tasks/createmktask.cpp     [License: GPL (v2+)]
A  +56   -0    utilities/assistants/panorama/tasks/createmktask.h     [License: GPL (v2+)]
A  +119  -0    utilities/assistants/panorama/tasks/createpreviewtask.cpp     [License: GPL (v2+)]
A  +61   -0    utilities/assistants/panorama/tasks/createpreviewtask.h     [License: GPL (v2+)]
A  +220  -0    utilities/assistants/panorama/tasks/createptotask.cpp     [License: GPL (v2+)]
A  +65   -0    utilities/assistants/panorama/tasks/createptotask.h     [License: GPL (v2+)]
A  +84   -0    utilities/assistants/panorama/tasks/huginexecutortask.cpp     [License: GPL (v2+)]
A  +55   -0    utilities/assistants/panorama/tasks/huginexecutortask.h     [License: GPL (v2+)]
A  +78   -0    utilities/assistants/panorama/tasks/optimisationtask.cpp     [License: GPL (v2+)]
A  +56   -0    utilities/assistants/panorama/tasks/optimisationtask.h     [License: GPL (v2+)]
A  +58   -0    utilities/assistants/panorama/tasks/panotask.cpp     [License: GPL (v2+)]
A  +65   -0    utilities/assistants/panorama/tasks/panotask.h     [License: GPL (v2+)]
A  +40   -0    utilities/assistants/panorama/tasks/panotasks.h     [License: GPL (v2+)]
A  +225  -0    utilities/assistants/panorama/tasks/preprocesstask.cpp     [License: GPL (v2+)]
A  +70   -0    utilities/assistants/panorama/tasks/preprocesstask.h     [License: GPL (v2+)]
M  +3    -0    utilities/kdesupport/kipi/kipipluginloader.cpp

http://commits.kde.org/digikam/1a5a6ea49710bed310875900814be70744385aa5