Opened 5 years ago
Closed 4 years ago
#15355 closed defect (fixed)
representation of CIF-element plus SR-element
Reported by: | dkrenn | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-6.4 |
Component: | symbolics | Keywords: | SR, CIF, representation, sd67, beginner |
Cc: | Merged in: | ||
Authors: | Daniel Krenn | Reviewers: | Ralf Stephan |
Report Upstream: | N/A | Work issues: | |
Branch: | c03f6bd (Commits) | Commit: | c03f6bd762293af1a1a6946bb2e395db98dfc9dc |
Dependencies: | Stopgaps: |
Description (last modified by )
We have
sage: x + CIF(RIF(-2,2), 0) x
and
sage: x + CIF(RIF(-2,2), RIF(-2,2)) x
which are wrong representation strings. Anyhow, the result themselves are correct since, for example, we have
sage: z = (x + CIF(RIF(-2,2),RIF(-2,2))) sage: z - x 0.?e1 + 0.?e1*I
In contrast, the following things work:
sage: x + RIF(-2,2) x + 0.?e1 sage: x + CIF(RIF(3.14,3.15), RIF(3.14, 3.15)) x + 3.15? + 3.15?*I sage: CIF(RIF(-2,2), RIF(-2,2)) 0.?e1 + 0.?e1*I sage: x + CIF(RIF(3.14,3.15), 0) x + 3.15?
Change History (9)
comment:1 Changed 5 years ago by
- Description modified (diff)
comment:2 Changed 5 years ago by
- Milestone changed from sage-6.1 to sage-6.2
comment:3 Changed 5 years ago by
- Milestone changed from sage-6.2 to sage-6.3
comment:4 Changed 5 years ago by
- Milestone changed from sage-6.3 to sage-6.4
comment:5 Changed 4 years ago by
- Branch set to u/dkrenn/t/15355
comment:6 Changed 4 years ago by
- Commit set to c03f6bd762293af1a1a6946bb2e395db98dfc9dc
- Keywords sd67 added
- Status changed from new to needs_review
comment:7 Changed 4 years ago by
- Keywords beginner added
comment:8 Changed 4 years ago by
- Reviewers set to Ralf Stephan
- Status changed from needs_review to positive_review
Is fine, thanks.
comment:9 Changed 4 years ago by
- Branch changed from u/dkrenn/t/15355 to c03f6bd762293af1a1a6946bb2e395db98dfc9dc
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
Seems to be fixed (tested in 6.6). Added doctest.
New commits:
adding doctests to check that 15355 is fixed