Opened 7 years ago

Closed 7 years ago

#1345 closed defect (fixed)

[with patch, with positive review] I is sometimes wrapped in SymbolicConstant, sometimes not

Reported by: cwitty Owned by: mhansen
Priority: major Milestone: sage-2.8.15
Component: calculus Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

This behavior seems strange:

 sage: foo = I+I
 sage: foo._operands
 [I, I]
 sage: [type(i) for i in foo._operands]
 [<class 'sage.calculus.calculus.SymbolicConstant'>,
  <class 'sage.functions.constants.I_class'>]

And here's another strange thing (probably the same bug):

 sage: is_SymbolicExpression(SR(I))
 False

Attachments (1)

1345.patch (3.7 KB) - added by mhansen 7 years ago.

Download all attachments as: .zip

Change History (6)

Changed 7 years ago by mhansen

comment:1 Changed 7 years ago by mhansen

  • Status changed from new to assigned
  • Summary changed from I is sometimes wrapped in SymbolicConstant, sometimes not to [with patch] I is sometimes wrapped in SymbolicConstant, sometimes not

comment:2 Changed 7 years ago by mhansen

  • Milestone changed from sage-2.9 to sage-2.8.15

comment:3 Changed 7 years ago by mhansen

This should be applied after #847.

comment:4 Changed 7 years ago by cwitty

  • Summary changed from [with patch] I is sometimes wrapped in SymbolicConstant, sometimes not to [with patch, with positive review] I is sometimes wrapped in SymbolicConstant, sometimes not

The code looks good and the doctests in the affected files pass. I approve.

comment:5 Changed 7 years ago by mabshoff

  • Resolution set to fixed
  • Status changed from assigned to closed

Merged in 2.8.15.alpha0.

Note: See TracTickets for help on using tickets.