patchAxiom Computer Algebra System - Patches: patch #3127, bug #9297 -- output misses...


You are not allowed to post comments on this tracker with your current authentication level.

patch #3127: bug #9297 -- output misses parenthesis in COMBF

Submitter:  Martin Rubey <kratt6>
Submitted:  Sat 12 Jun 2004 04:05:32 PM UTC
Category:  None Priority:  1 - Later
Status:  None Privacy:  Public
Assigned to:  billpage1 Open/Closed:  Open

Tue 28 Dec 2004 02:31:02 PM UTC, comment #1: 

What is the status of this patch. Does it fix the general case?

Bill Page <billpage1>
Group Member
Sat 12 Jun 2004 04:05:32 PM UTC, original submission:  

From William Sit:

The fix is not as easy as it seems. It is not clear whether the "bug" should be fixed locally in combfunc.spad (a simple, temporary, but unsatisfactory fix is to modify ddprod and ddsum with paren(...)) with limited effect, or globally in OutputForm (outform.spad), where the bug was perhaps recognized:

    -- Todo:
    -- bug in product, paren blankSeparate []

The temperary fix is unsatisfactory because we should not SIMPLY add parenthesis to EVERY summation or product.  This is a general problem and requires an examination of the context to decide whether the parenthesis is necessary and that may be subjective. (I find controlling the output in Mathematica equally difficult, but generally, Mathematica did a much better job -- except in cases I
don't like :-). Since OutputForm is built bottom up (the responsibility of each domain via coerce), it may be difficult to make the decision even if it can be done objectively.

Martin Rubey <kratt6>


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

Attached Files
file #7547:  combfout.patch added by kratt6 (648B - text/x-diff)


Depends on the following items: None found

Items that depend on this one: None found


CC list is empty


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.


Follow 2 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2004-12-28 billpage1 Assigned toNone billpage1
2004-06-12 kratt6 Attached File- Added combfout.patch, #3357

Back to the top

Powered by Savane 3.13-4b48.
Corresponding source code