autopano null value at MultiMatch.LoadKeysets

Bug #162575 reported by Boyan Penkov
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
hugin (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: autopano-sift

boyan@boyan-desktop:~/Downloads$ autopano /f "/home/boyan/Downloads/DSLC12133235.JPG" "/home/boyan/Downloads/DSLC12133300.JPG"
autopano-sift, Automatic panorama generation program

Loading keyfiles
ex: System.Xml.XmlException: Text node cannot appear in this state. Line 1, position 1.
  at System.Xml.XmlTextReader.ReadText (Boolean notWhitespace) [0x00000]
  at System.Xml.XmlTextReader.ReadContent () [0x00000]
  at System.Xml.XmlTextReader.Read () [0x00000]
  at System.Xml.XmlReader.MoveToContent () [0x00000]
  at System.Xml.Serialization.XmlSerializationReaderInterpreter.ReadRoot () [0x00000]
  at System.Xml.Serialization.XmlSerializer.Deserialize (System.Xml.Serialization.XmlSerializationReader reader) [0x00000]

Unhandled Exception: System.NullReferenceException: A null value was found where an object instance was required.
  at MultiMatch.LoadKeysets (System.String[] filenames) [0x00000]
  at Autopano.Main (System.String[] args) [0x00000]

Related branches

CVE References

Revision history for this message
Francesco Potortì (pot) wrote :

autopano reads in text files with dot posistions, not images.
Anyway, it should output a comprehensible error message rather than a stack trace.

Revision history for this message
Boyan Penkov (boyan-penkov) wrote : Re: [Bug 162575] Re: autopano null value at MultiMatch.LoadKeysets

Thanks! I think I'm in a bit over my head, then. Can you point me to a
tutorial? ;)

BIP!!!

On Tue, 2008-02-05 at 11:48 +0000, Francesco Potortì wrote:

> autopano reads in text files with dot posistions, not images.
> Anyway, it should output a comprehensible error message rather than a stack trace.
>

-----
Boyan Penkov

Revision history for this message
Sybren Stüvel (sybren-stuvel) wrote :

I got the same error, from an invocation of autopano-sift by Hugin.

In the package autopano-sift (2.4-0ubuntu8), the executable is called autopano. Using the default settings, Hugin (0.7.0~svn3191+beta5-1ubuntu1) expects it to be called autopano-sift. After changing Hugin's preferences, it calls:

autopano --maxmatches 20 /tmp/ap_resjyB4Iq "/home/sybren/2008/20080926-Utrecht/hdr1/jpeg/hdr1-1.jpg" "/home/sybren/2008/20080926-Utrecht/hdr1/jpeg/hdr1-2.jpg" "/home/sybren/2008/20080926-Utrecht/hdr1/jpeg/hdr1-3.jpg" "/home/sybren/2008/20080926-Utrecht/hdr1/jpeg/hdr1-4.jpg" "/home/sybren/2008/20080926-Utrecht/hdr1/jpeg/hdr1-5.jpg" "/home/sybren/2008/20080926-Utrecht/hdr1/jpeg/hdr1-6.jpg" "/home/sybren/2008/20080926-Utrecht/hdr1/jpeg/hdr1-7.jpg"

The program output reads:

autopano-sift, Automatic panorama generation program

Loading keyfiles
ex: System.Xml.XmlException: Text node cannot appear in this state. Line 1, position 1.
  at System.Xml.XmlTextReader.ReadText (Boolean notWhitespace) [0x00000]
  at System.Xml.XmlTextReader.ReadContent () [0x00000]
  at System.Xml.XmlTextReader.Read () [0x00000]
  at System.Xml.XmlReader.MoveToContent () [0x00000]
  at System.Xml.Serialization.XmlSerializationReaderInterpreter.ReadRoot () [0x00000]
  at System.Xml.Serialization.XmlSerializer.Deserialize (System.Xml.Serialization.XmlSerializationReader reader) [0x00000]

Unhandled Exception: System.NullReferenceException: Object reference not set to an instance of an object
  at MultiMatch.LoadKeysets (System.String[] filenames) [0x00000]
  at Autopano.Main (System.String[] args) [0x00000]

I would expect the default Hugin preferences to simply work with the Ubuntu packages.

Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

I also had this error executing autopano from hugin, looking at the log, but I received a message from Hugin saying the I had to define some matching points between the 2 images.
I defined them, and it was working.

So this error is due to a bad use of autopano.

And about the default value in hugin, I'm waiting for the debian package of final 0.7.0 to make the 'Ubuntu' version of it (renaming autopano-SIFT by default).

Changed in autopano-sift:
status: New → Confirmed
Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

I've checked the online help of autopano, and it should be able to resolve automatically the mathcing points.

Will investigate.

Changed in autopano-sift:
assignee: nobody → fabricesp
status: Confirmed → In Progress
Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

Got it!
Correct parameters for autopano-SIFT command in hugin is:
executable: autopano-complete
Parameters: --points %p -o %o %i

Changing package to hugin one

Revision history for this message
Boyan Penkov (boyan-penkov) wrote :

Excellent! I'm currently away from the workstation I use autopano on;
as soon as I have access again, I'll confirm the solution.

Thanks much!

On Thu, 2008-11-06 at 05:51 +0000, fabrice_sp wrote:

> Got it!
> Correct parameters for autopano-SIFT command in hugin is:
> executable: autopano-complete
> Parameters: --points %p -o %o %i
>
> Changing package to hugin one
>
> ** Changed in: hugin (Ubuntu)
> Sourcepackagename: autopano-sift => hugin
>

-----
Boyan Penkov

Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

Wating for confirmation that those parameters actually fixe the problem.

Changed in hugin:
status: In Progress → Incomplete
Revision history for this message
eSe (snaker) wrote :

I can confirm thar those parameters fixed the problem in ubuntu intrepid i386

Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

This change will be delivered with hugin 7.0 final version

Changed in hugin:
status: Incomplete → In Progress
Revision history for this message
Boyan Penkov (boyan-penkov) wrote :

After a ridiculously lengthy delay (I was far from home on business), I
too can confirm this works on 8.10 AMD64.

Thanks guys!

On Mon, 2008-11-10 at 05:25 +0000, fabrice_sp wrote:

> This change will be delivered with hugin 7.0 final version
>
> ** Changed in: hugin (Ubuntu)
> Status: Incomplete => In Progress
>

-----
Boyan Penkov

Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

This bug will be closed when bug #285674 will be closed.
Putting back in Confirmed state.

Changed in hugin:
assignee: fabricesp → nobody
status: In Progress → Confirmed
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (7.8 KiB)

This bug was fixed in the package hugin - 0.7.0-1ubuntu1

---------------
hugin (0.7.0-1ubuntu1) jaunty; urgency=low

  * Closes LP: #285674
  * Closes LP: #162575
    - debian/control: added dependency on autopano-sift
    - debian/patches: added 91_ubuntu-autopano-sift to fix default config
  * debian/control: change build dependency from libwxgtk2.6-dev to
    libwxgtk2.8-dev

hugin (0.7.0-1) experimental; urgency=low

  * New upstream release (Closes: #502721).
  * Depend on both enblend (>= 3.2) and enfuse, now that they are
    available (Closes: #495874).
  * Drop the following patch accordingly:
     - 50_disable_enblend_compression
  * debian/rules:
     - Specify “ChangeLog” in the dh_installchangelogs call since that's a
       proper release and no longer an SVN snapshot.
     - Get rid of the autooptimi{s,z}er renaming hack, fixed upstream.

hugin (0.7.0~svn3191+beta5-2) unstable; urgency=low

  * Update README.Debian to reflect the current status of hugin with
    respect to enblend/enfuse. Also point to the following page, which
    will contain uptodate information about the backports for the lenny
    lifetime:
      http://pkg-phototools.alioth.debian.org/lenny.html
  * Add forgotten “tca_correct” binary to debian/hugin-tools.install,
    thanks to Chris Chiappa (Closes: #502104). Another one is
    “matchpoint”, but since it wasn't requested yet, don't touch it at the
    moment.

hugin (0.7.0~svn3191+beta5-1) unstable; urgency=low

  * New upstream release candidate: 0.7.0_beta5. The exact revision wasn't
    specified but that's mostly the same as svn r3191.
  * Adjust dependencies, thanks to several reporters, and especially to
    Andreas Metzler for the checks on enblend and the pointers:
     - Add make to hugin's Depends (Closes: #492469).
     - Move enblend from hugin-tools's Recommends to hugin's Depends, as
       it is now a requirement (Closes: #394647, #491539).
     - Also add libimage-exiftool-perl since upstream specifies that the
       the exiftool binary (from the Image::ExifTool Perl module) is
       needed.
     - Version the Build-Depends on libexiv2-dev: “>= 0.12”. With former
       versions, an FTBFS is triggered due to undefined references to
       Jpeg-related functions (Closes: #491335).
  * Add autopano-sift-c as alternative to autopano-sift to the suggested
    packages. As a reminder, patent issues prevent the inclusion of those
    packages in Debian.
  * Fix manpage mismatch: /usr/bin/autooptimizer versus autooptimizer(1),
    (spotted by Julien Valroff, thanks):
     - debian/rules: Copy generated doc/autooptimizer.1 into
       doc/autooptimiser.1, and remember it for cleanup.
     - debian/hugin-tools.manpages: Replace the z version with the s one.
  * Disable enblend compression, since Debian has enblend 3.0 for now and
    this option would require enblend 3.1 or a CVS snapshot. That means
    that the compression setting is going to be silently ignored for now,
    thanks to a quick patch by Steve Cotton (Closes: #491227, #491562):
     - 50_disable_enblend_compression: Do not call enblend with
       a “--compression” argument.
  * Add a README.Debian file to document that some fea...

Read more...

Changed in hugin:
status: Confirmed → Fix Released
Revision history for this message
Boyan Penkov (boyan-penkov) wrote :
Download full text (8.4 KiB)

Agreed, fixed.

Thanks to all involved!

On Sun, 2008-11-30 at 19:20 +0000, Launchpad Bug Tracker wrote:

> This bug was fixed in the package hugin - 0.7.0-1ubuntu1
>
> ---------------
> hugin (0.7.0-1ubuntu1) jaunty; urgency=low
>
> * Closes LP: #285674
> * Closes LP: #162575
> - debian/control: added dependency on autopano-sift
> - debian/patches: added 91_ubuntu-autopano-sift to fix default config
> * debian/control: change build dependency from libwxgtk2.6-dev to
> libwxgtk2.8-dev
>
> hugin (0.7.0-1) experimental; urgency=low
>
> * New upstream release (Closes: #502721).
> * Depend on both enblend (>= 3.2) and enfuse, now that they are
> available (Closes: #495874).
> * Drop the following patch accordingly:
> - 50_disable_enblend_compression
> * debian/rules:
> - Specify “ChangeLog” in the dh_installchangelogs call since that's a
> proper release and no longer an SVN snapshot.
> - Get rid of the autooptimi{s,z}er renaming hack, fixed upstream.
>
> hugin (0.7.0~svn3191+beta5-2) unstable; urgency=low
>
> * Update README.Debian to reflect the current status of hugin with
> respect to enblend/enfuse. Also point to the following page, which
> will contain uptodate information about the backports for the lenny
> lifetime:
> http://pkg-phototools.alioth.debian.org/lenny.html
> * Add forgotten “tca_correct” binary to debian/hugin-tools.install,
> thanks to Chris Chiappa (Closes: #502104). Another one is
> “matchpoint”, but since it wasn't requested yet, don't touch it at the
> moment.
>
> hugin (0.7.0~svn3191+beta5-1) unstable; urgency=low
>
> * New upstream release candidate: 0.7.0_beta5. The exact revision wasn't
> specified but that's mostly the same as svn r3191.
> * Adjust dependencies, thanks to several reporters, and especially to
> Andreas Metzler for the checks on enblend and the pointers:
> - Add make to hugin's Depends (Closes: #492469).
> - Move enblend from hugin-tools's Recommends to hugin's Depends, as
> it is now a requirement (Closes: #394647, #491539).
> - Also add libimage-exiftool-perl since upstream specifies that the
> the exiftool binary (from the Image::ExifTool Perl module) is
> needed.
> - Version the Build-Depends on libexiv2-dev: “>= 0.12”. With former
> versions, an FTBFS is triggered due to undefined references to
> Jpeg-related functions (Closes: #491335).
> * Add autopano-sift-c as alternative to autopano-sift to the suggested
> packages. As a reminder, patent issues prevent the inclusion of those
> packages in Debian.
> * Fix manpage mismatch: /usr/bin/autooptimizer versus autooptimizer(1),
> (spotted by Julien Valroff, thanks):
> - debian/rules: Copy generated doc/autooptimizer.1 into
> doc/autooptimiser.1, and remember it for cleanup.
> - debian/hugin-tools.manpages: Replace the z version with the s one.
> * Disable enblend compression, since Debian has enblend 3.0 for now and
> this option would require enblend 3.1 or a CVS snapshot. That means
> that the compression setting is going to be silently ignored for ...

Read more...

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.