bugGNU Octave - Bugs: bug #51781, [octave forge] (miscellaneous)...

 
 

bug #51781: [octave forge] (miscellaneous) textable error on valid input

Submitter:  andre <wienerschnitzel>
Submitted:  Fri 18 Aug 2017 06:05:55 AM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Unexpected Error or Warning
Status:  Fixed Assigned to:  juanpi
Originator Name:  Open/Closed:  * Closed
Release:  * 4.0.0 Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Sat 19 Aug 2017 03:27:58 PM UTC, comment #5: 

Fixed this in dev and also changed a call to octave_config_info for a call to _have_feature_



This problem has been fixed in the development version. The fix
will be available in the next major software release. Thank you
for your bug report.

Juan Pablo Carbajal <juanpi>
Group Member
Fri 18 Aug 2017 04:19:47 PM UTC, comment #4: 

Sorry, reopening.

The package should now drop its dependency on the "general" package, because it looks to me like inputParser was the only reason for that dependency.

Mike Miller <mtmiller>
Group Member
Fri 18 Aug 2017 04:18:48 PM UTC, comment #3: 

Yes it looks fixed to me.

http://hg.code.sf.net/p/octave/miscellaneous/rev/975ac6a91db5

Unit tests would be nice also.

For the OP - you can grab the updated textable.m from the hg repo (http://hg.code.sf.net/p/octave/miscellaneous/file/54f303dc03ec/inst/textable.m) if you don't want to wait for a new release of the miscellaneous package.

Mike Miller <mtmiller>
Group Member
Fri 18 Aug 2017 09:39:09 AM UTC, comment #2: 

I pushed a change to the dev version that fixed the issue.
Can anyone check.
You can use the simple demo that I added (OP test case).

Juan Pablo Carbajal <juanpi>
Group Member
Fri 18 Aug 2017 06:26:26 AM UTC, comment #1: 

Confirmed, the error occurs because the textable function is expecting the old incompatible form of inputParser:


111:   p = p.addSwitch ("clines");


The function will need to be updated to use Octave's built in inputParser, and preferably have a test case added to it.

Mike Miller <mtmiller>
Group Member
Fri 18 Aug 2017 06:05:55 AM UTC, original submission:  

GNU Octave Version: 4.0.0
GNU Octave License: GNU General Public License
Operating System: Linux 4.8.0-52-generic #55~16.04.1-Ubuntu SMP Fri Apr 28 14:36:29 UTC 2017 x86_64
----------------------------------------------------------------------
Package Name   | Version | Installation directory
---------------+---------+-----------------------
      general *|   2.0.0 | /usr/share/octave/packages/general-2.0.0
miscellaneous *|   1.2.1 | /usr/share/octave/packages/miscellaneous-1.2.1

Steps:



A =

   1   2   3
   4   5   6

>> textable(A)
error: value on right hand side of assignment is undefined
error: called from
    textable at line 111 column 5


result: textable reports error on valid argument.
expected: textable should not report an error given valid input and produce latex code.

andre <wienerschnitzel>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by juanpi (Posted a comment)
  • -email is unavailable- added by mtmiller (Posted a comment)
  • -email is unavailable- added by wienerschnitzel (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 11 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-08-19 juanpi StatusIn Progress Fixed
        Open/ClosedOpen Closed
    2017-08-18 mtmiller StatusFixed In Progress
        Open/ClosedClosed Open
    2017-08-18 mtmiller StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2017-08-18 juanpi StatusConfirmed Ready For Test
        Assigned toNone juanpi
    2017-08-18 mtmiller CategoryOctave Function Octave Package
        StatusNone Confirmed
        Summarytextable error on valid input [octave forge] (miscellaneous) textable error on valid input

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code