Opened 15 years ago
Closed 15 years ago
#1816 closed enhancement (wontfix)
[with patch, with negative review] rename MPolynomialRing.repr_long method to __str__
Reported by: | Martin Albrecht | Owned by: | Martin Albrecht |
---|---|---|---|
Priority: | trivial | Milestone: | sage-duplicate/invalid/wontfix |
Component: | commutative algebra | Keywords: | |
Cc: | Merged in: | ||
Authors: | Reviewers: | ||
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
Just as symbolic variables behave (and what is the Python-way IIRC):
sage: f = x/var('y') sage: f x/y sage: str(f) ' x\r\n -\r\n y' sage: print str(f) x - y
Attachments (1)
Change History (6)
Changed 15 years ago by
Attachment: | trac_1816.patch added |
---|
comment:1 Changed 15 years ago by
Summary: | rename MPolynomialRing.repr_long method to __str__ → [with patch, needs review] rename MPolynomialRing.repr_long method to __str__ |
---|
comment:2 Changed 15 years ago by
Summary: | [with patch, needs review] rename MPolynomialRing.repr_long method to __str__ → [with patch, with negative review] rename MPolynomialRing.repr_long method to __str__ |
---|
comment:3 Changed 15 years ago by
Because I disagree with Nick's verdict, I forwarded this to [sage-devel]:
http://groups.google.com/group/sage-devel/browse_thread/thread/612b3ec4a61310fa
I figure, that this is more a design choice than a correctness issue and thus it should be discussed on [sage-devel] rather than here. I hope that's okay with you, Nick.
comment:4 Changed 15 years ago by
My impression is: The verdict on [sage-devel] was overall negative, so I propose to close this ticket as wontfix
.
comment:5 Changed 15 years ago by
Milestone: | sage-3.0 → sage-duplicate/invalid |
---|---|
Resolution: | → wontfix |
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
The patch is fine, and does what it says, but it makes it look like printing a polynomial ring will give this verbose output:
That's *not* okay -- way too much by default!