Opened 8 years ago

Last modified 5 years ago

#11764 new enhancement

define the class MPolynomial at the top-level

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

Description

for univariate polynomials we have the class Polynomial:

sage: R.<x> = QQ[]
sage: isinstance(x+1, Polynomial)
True

However for multivariate polynomials we have to write:

sage: R.<x,y> = QQ[]
sage: isinstance(x+y, sage.rings.polynomial.multi_polynomial.MPolynomial)
True

I suggest MPolynomial is defined as an alias for sage.rings.polynomial.multi_polynomial.MPolynomial so that we can simply write:

sage: R.<x,y> = QQ[]
sage: isinstance(x+y, MPolynomial)
True

Change History (7)

comment:1 follow-up: Changed 8 years ago by malb

I think this needs to be discussed on [sage-devel]. Many people are pushing for less stuff in the global namespace so we cannot just add stuff without asking first.

comment:2 in reply to: ↑ 1 Changed 8 years ago by zimmerma

  • Cc mmezzarobba added

Replying to malb:

I think this needs to be discussed on [sage-devel]. Many people are pushing for less stuff in the global namespace so we cannot just add stuff without asking first.

Martin, feel free to forward to sage-devel, since I am not subscribed.

On the other hand for coherence one could replace Polynomial by sage.rings.polynomial.polynomial_ring.Polynomial.

Paul

comment:4 Changed 6 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:5 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:6 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:7 Changed 5 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4
Note: See TracTickets for help on using tickets.