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.

Sign up for GitHub

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

Jump to bottom

Polynomial_singular_repr is pretty messed up #2331

Closed
malb opened this issue Feb 27, 2008 · 3 comments
Closed

Polynomial_singular_repr is pretty messed up #2331

malb opened this issue Feb 27, 2008 · 3 comments
Assignees
@malb
Labels
c: interfaces r: invalid t: enhancement

Comments

@malb
Copy link
Member

malb commented Feb 27, 2008

  • the superclass injection trick is a bad design decision
  • derivatives don't need to be handled by Singular
  • the have_ring option should go, because we don't need this dirty speed improvement anymore (we have libSingular)

Component: interfaces

Reviewer: Vincent Delecroix

Issue created by migration from https://trac.sagemath.org/ticket/2331
@malb malb added this to the sage-5.11 milestone Feb 27, 2008
@malb malb self-assigned this Feb 27, 2008
@malb
Copy link
Member Author

malb commented Jan 22, 2009

comment:2

This is an enhancement.

@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
@kwankyu
Copy link
Collaborator

kwankyu commented Jun 5, 2022

comment:7

Item (3) is handled in #33949. The objectives of the other items are not clear and perhaps not valid after 14 years.

@kwankyu kwankyu removed this from the sage-6.4 milestone Jun 5, 2022
@videlec
Copy link
Contributor

videlec commented Jun 16, 2022

Reviewer: Vincent Delecroix

@mkoeppe mkoeppe closed this as completed Aug 2, 2022
@fchapoton fchapoton mentioned this issue Jun 12, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@malb malb

Labels
c: interfaces r: invalid t: enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@malb @kwankyu @jdemeyer @videlec @mkoeppe