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


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

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

Submitted by:  Martin Rubey <kratt6>
Submitted on:  Sat Jun 12 16:05:32 2004  
Category: NonePriority: 1 - Later
Status: NonePrivacy: Public
Assigned to: Bill Page <billpage1>Open/Closed: Open

Tue Dec 28 14:31:02 2004, comment #1:

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

Bill Page <billpage1>
Project MemberIn charge of this item.
Sat Jun 12 16:05:32 2004, 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>


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


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

Only logged-in users can vote.


Please enter the title of George Orwell's famous dystopian book (it's a date):



Follow 2 latest changes.

Date Changed By Updated Field Previous Value => Replaced By
Tue Dec 28 14:31:02 2004billpage1Assigned toNone=>billpage1
Sat Jun 12 16:05:32 2004kratt6Attached File-=>Added combfout.patch, #3357

Back to the top

Powered by Savane 3.1-cleanup1