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

Make .variety() work for ideals of univariate polynomial rings #31331

Open
sagetrac-monniaux mannequin opened this issue Feb 3, 2021 · 3 comments
Open

Make .variety() work for ideals of univariate polynomial rings #31331

sagetrac-monniaux mannequin opened this issue Feb 3, 2021 · 3 comments

Comments

@sagetrac-monniaux
Copy link
Mannequin

sagetrac-monniaux mannequin commented Feb 3, 2021

Sage 9.2:

sage: PR = PolynomialRing(QQ, ['x'])
sage: I = ideal(PR('x^2 - 4'))
sage: print(I.variety())

Traceback (most recent call last):
  File "crash.sage.py", line 8, in <module>
    print(I.variety())
  File "sage/structure/element.pyx", line 493, in sage.structure.element.Element.__getattr__ (build/cythonized/sage/structure/element.c:4703)
  File "sage/structure/element.pyx", line 506, in sage.structure.element.Element.getattr_from_category (build/cythonized/sage/structure/element.c:4815)
  File "sage/cpython/getattr.pyx", line 372, in sage.cpython.getattr.getattr_from_other_class (build/cythonized/sage/cpython/getattr.c:2619)
AttributeError: 'Ideal_1poly_field' object has no attribute 'variety'

And I had another, funkier error attempting the same thing from the notebook.


/opt/SageMath/9.2/local/lib/python3.8/site-packages/sage/structure/element.pyx in sage.structure.element.Element.__getattr__ (build/cythonized/sage/structure/element.c:4703)()
    491             AttributeError: 'LeftZeroSemigroup_with_category.element_class' object has no attribute 'blah_blah'
    492         """
--> 493         return self.getattr_from_category(name)
    494 
    495     cdef getattr_from_category(self, name):

/opt/SageMath/9.2/local/lib/python3.8/site-packages/sage/structure/element.pyx in sage.structure.element.Element.getattr_from_category (build/cythonized/sage/structure/element.c:4815)()
    504         else:
    505             cls = P._abstract_element_class
--> 506         return getattr_from_other_class(self, cls, name)
    507 
    508     def __dir__(self):

CC: @slel

Component: commutative algebra

Keywords: ideal, variety, polynomial

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

@sagetrac-monniaux sagetrac-monniaux mannequin added this to the sage-9.3 milestone Feb 3, 2021
@sagetrac-monniaux sagetrac-monniaux mannequin added the p: major / 3 label Feb 3, 2021
@slel

This comment has been minimized.

@slel
Copy link
Member

slel commented Feb 5, 2021

Changed keywords from none to ideal, variety, polynomial

@slel slel changed the title .variety() does not work for ideal of monovariate polynomials Make .variety() work for ideals of univariate polynomial rings Feb 5, 2021
@mkoeppe
Copy link
Member

mkoeppe commented Mar 24, 2021

comment:2

Sage development has entered the release candidate phase for 9.3. Setting a new milestone for this ticket based on a cursory review of ticket status, priority, and last modification date.

@mkoeppe mkoeppe modified the milestones: sage-9.3, sage-9.4 Mar 24, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.4, sage-9.5 Jul 19, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.5, sage-9.6 Dec 18, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.6, sage-9.7 May 3, 2022
@mkoeppe mkoeppe modified the milestones: sage-9.7, sage-9.8 Aug 31, 2022
@mkoeppe mkoeppe removed this from the sage-9.8 milestone Jan 29, 2023
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

2 participants