Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

define the class MPolynomial at the top-level #11764

Open
zimmermann6 opened this issue Aug 31, 2011 · 3 comments
Open

define the class MPolynomial at the top-level #11764

zimmermann6 opened this issue Aug 31, 2011 · 3 comments

Comments

@zimmermann6
Copy link

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

CC: @mezzarobba

Component: commutative algebra

Issue created by migration from https://trac.sagemath.org/ticket/11764

@malb
Copy link
Member

malb commented Aug 31, 2011

comment:1

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.

@zimmermann6
Copy link
Author

comment:2

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

@malb
Copy link
Member

malb commented Aug 31, 2011

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@mkoeppe mkoeppe removed this from the sage-6.4 milestone Dec 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants