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

univariate homogenize is not the same as multivariate homogenize #2352

Closed
ncalexan mannequin opened this issue Feb 29, 2008 · 13 comments
Closed

univariate homogenize is not the same as multivariate homogenize #2352

ncalexan mannequin opened this issue Feb 29, 2008 · 13 comments

Comments

@ncalexan
Copy link
Mannequin

ncalexan mannequin commented Feb 29, 2008

Ticket #2349 fixes the multivariate case. This ticket is for the univariate case. Some examples (these work after like this AFTER #2349 has been applied):

sage: x = Zmod(3)['x'].0; (x^2 + x).homogenize()
---------------------------------------------------------------------------
<type 'exceptions.AttributeError'>        Traceback (most recent call last)

/Users/ncalexan/<ipython console> in <module>()

<type 'exceptions.AttributeError'>: 'sage.rings.polynomial.polynomial_modn_dense_ntl.Po' object has no attribute 'homogenize'
sage: x = PolynomialRing(Zmod(3), 1, 'x').0; (x^2 + x).homogenize()
x^2 + x*h
sage: x = GF(3)['x'].0; (x^2 + x).homogenize()
---------------------------------------------------------------------------
<type 'exceptions.AttributeError'>        Traceback (most recent call last)

/Users/ncalexan/<ipython console> in <module>()

<type 'exceptions.AttributeError'>: 'sage.rings.polynomial.polynomial_modn_dense_ntl.Po' object has no attribute 'homogenize'
sage: x = PolynomialRing(GF(3), 1, 'x').0; (x^2 + x).homogenize()
x^2 + x*h

CC: @ncalexan

Component: commutative algebra

Keywords: univariate polynomial homogenize

Author: Luis Felipe Tabera Alonso, Julian Rueth

Branch/Commit: 03bf755

Reviewer: Julian Rueth, Luis Felipe Tabera Alonso

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

@ncalexan ncalexan mannequin added this to the sage-5.11 milestone Feb 29, 2008
@ncalexan ncalexan mannequin assigned malb Feb 29, 2008
@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
@lftabera
Copy link
Contributor

Author: Luis Felipe Tabera Alonso

@lftabera
Copy link
Contributor

comment:5

One step more to unify some methods of univariate and multivariate polynomials.


New commits:

e5599eaTicket 2352: univariate homogenize

@lftabera
Copy link
Contributor

@lftabera
Copy link
Contributor

Commit: e5599ea

@saraedum
Copy link
Member

comment:6

Should (1+t).homogenize('t') really be zero? The docstring says that setting this variable to one yields the original polynomial which does not seem to be the case. So should we clarify (also in the multivariate case) that strange things may happen if var is a variable which appears in the polynomial?

@saraedum
Copy link
Member

@saraedum
Copy link
Member

Changed commit from e5599ea to 03bf755

@saraedum
Copy link
Member

Reviewer: Julian Rueth

@saraedum
Copy link
Member

Changed author from Luis Felipe Tabera Alonso to Luis Felipe Tabera Alonso, Julian Rueth

@saraedum
Copy link
Member

comment:10

I tried to clarify the behaviour in the docstrings. I hope you do not mind my changes.


New commits:

03bf755Improved docstrings of homogenize() of polynomials

@lftabera
Copy link
Contributor

comment:11

Thanks, the documentation has improved, much clearer now.

@lftabera
Copy link
Contributor

Changed reviewer from Julian Rueth to Julian Rueth, Luis Felipe Tabera Alonso

@vbraun
Copy link
Member

vbraun commented Jul 29, 2014

Changed branch from u/saraedum/ticket/2352 to 03bf755

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

5 participants