bugGNU Octave - Bugs: bug #39345, symbolic package: Fix test in...


bug #39345: symbolic package: Fix test in sym2poly

Submitted by:  Rafael Laboissiere <rlaboiss>
Submitted on:  Wed 26 Jun 2013 04:06:00 PM UTC  
Category:  Octave Forge Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Wont Fix Assigned to:  None
Originator Name:  Open/Closed:  Closed
Release:  dev Operating System:  GNU/Linux

Add a New Comment(Rich Markup)

You are not logged in

Please log in, so followups can be emailed to you.


Sun 22 Feb 2015 06:20:02 AM UTC, comment #3:

The brand new symbolic package does not have a failing test in sym2poly:

Closing report as won't fix since the old symbolic package has been replaced.

Mike Miller <mtmiller>
Project Administrator
Wed 26 Jun 2013 08:12:47 PM UTC, comment #2:

You are right, the problem is not due to the fact that the arguments are cell arrays, but because the elements are symbolic expressions. For instance, this fails:

Anyway, the patch is still necessary. However, if you decide to apply it, please change the description.

Rafael Laboissiere <rlaboiss>
Wed 26 Jun 2013 05:52:44 PM UTC, comment #1:

I don't have the symbolic package to test this but does assert really fails on cell arrays? The following works fine on my system (dev version and 3.6.2).

Carnë Draug <carandraug>
Project Member
Wed 26 Jun 2013 04:06:00 PM UTC, original submission:

This bug was introduced by me in cset 46a454. The assert function does not support cells as arguments to be compared (observed and expected). This can be fixed by passing the string representation of both values, as in the patched attached to this bug report.

Rafael Laboissiere <rlaboiss>


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

Attach Files:

Attached Files
file #28427:  fix-test-in-sym2poly.diff added by rlaboiss (875B - text/x-patch - Patch for inst/sym2poly.m)


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -email is unavailable- added by mtmiller (Updated the item)
  • -email is unavailable- added by rlaboiss (Submitted the item)

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.




    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-02-22 mtmiller StatusPatch Submitted => Wont Fix
        Open/ClosedOpen => Closed
    2014-01-20 mtmiller StatusNone => Patch Submitted
    2013-06-26 carandraug SummaryFix test in sym2poly => symbolic package: Fix test in sym2poly
    2013-06-26 rlaboiss Attached File- => Added fix-test-in-sym2poly.diff, #28427

    Back to the top

    Powered by Savane 3.3