Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#2042 closed defect (duplicate)

[with patch] change is_simplified to has_been_simplified in calculus.py (proper subset of #2073)

Reported by: AlexGhitza Owned by: AlexGhitza
Priority: minor Milestone: sage-duplicate/invalid/wontfix
Component: calculus Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

See http://groups.google.com/group/sage-devel/browse_thread/thread/e342c0b1020de8bc

The point of is_simplified() is to keep track of whether the expression has already been simplified, rather than to check whether the expression is simplified. The attached patch changes the name to has_been_simplified and adds a doctest.

Attachments (1)

2042-has_been_simplified.patch (6.3 KB) - added by AlexGhitza 13 years ago.

Download all attachments as: .zip

Change History (4)

Changed 13 years ago by AlexGhitza

comment:1 Changed 13 years ago by AlexGhitza

  • Summary changed from [with patch, needs review] change is_simplified to has_been_simplified in calculus.py to [with patch] change is_simplified to has_been_simplified in calculus.py (proper subset of #2073)

William's patches in #2073 already fix this and much much more. So close this ticket (as duplicate, I guess) as soon as #2073 gets closed.

comment:2 Changed 13 years ago by AlexGhitza

  • Resolution set to duplicate
  • Status changed from new to closed

comment:3 Changed 13 years ago by mabshoff

  • Milestone changed from sage-2.10.2 to sage-duplicate/invalid
Note: See TracTickets for help on using tickets.