Opened 15 years ago

Closed 15 years ago

#3138 closed task (fixed)

[with patch, positive review] Singular multivariate polynomial ring has redundant _repr_ method

Reported by: broune Owned by: broune
Priority: trivial Milestone: sage-3.0.2
Component: algebra Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

MPolynomialRing_libsingular in sage/rings/polynomial/multi_polynomial_libsingular.pyx defines a _repr_ method which does the same thing as the _repr_ method that it inherits from MPolynomialRing_generic in sage/rings/polynomial/multi_polynomial_ring_generic.pyx

Thus the _repr_ method is redundant and should be removed.

Attachments (1)

no_singular_repr.changeset (1.1 KB) - added by broune 15 years ago.

Download all attachments as: .zip

Change History (4)

Changed 15 years ago by broune

Attachment: no_singular_repr.changeset added

comment:1 Changed 15 years ago by broune

Status: newassigned

comment:2 Changed 15 years ago by malb

Summary: [with patch, needs review] Singular multivariate polynomial ring has redundant _repr_ method[with patch, positive review] Singular multivariate polynomial ring has redundant _repr_ method

Yep, I wrote the libsingular version before the generic one. Positive review.

comment:3 Changed 15 years ago by mabshoff

Resolution: fixed
Status: assignedclosed

Merged in Sage 3.0.2.alpha0

Note: See TracTickets for help on using tickets.