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

__floordiv__ for sparse polynomials #34653

Closed
mantepse opened this issue Oct 12, 2022 · 8 comments
Closed

__floordiv__ for sparse polynomials #34653

mantepse opened this issue Oct 12, 2022 · 8 comments
Labels
c: algebra p: major / 3 t: bug
Milestone
sage-9.8

Comments

@mantepse
Copy link
Collaborator

Before this ticket, the following would raise an ArithmeticError in the sparse case:

sage: P.<x,y> = PolynomialRing(SR)
sage: R.<z> = PolynomialRing(P, sparse=False); R
Univariate Polynomial Ring in z over Multivariate Polynomial Ring in x, y over Symbolic Ring
sage: (x*z) // x
z
sage: R.<z> = PolynomialRing(P, sparse=True); R
Sparse Univariate Polynomial Ring in z over Multivariate Polynomial Ring in x, y over Symbolic Ring
sage: (x*z) // x
z

Component: algebra

Author: Martin Rubey

Branch/Commit: 9577386

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/34653
@mantepse mantepse added this to the sage-9.8 milestone Oct 12, 2022
@mantepse
Copy link
Collaborator Author

Branch: u/mantepse/__floordiv___for_sparse_polynomials

@mantepse
Copy link
Collaborator Author

Commit: 9577386

@mantepse
Copy link
Collaborator Author

Author: Martin Rubey

@mantepse
Copy link
Collaborator Author

New commits:

9577386implement Polynomial_generic_sparse.__floordiv__

@mantepse

This comment has been minimized.

Sign in to view
@tscrim
Copy link
Collaborator

tscrim commented Oct 13, 2022

comment:3

LGTM.

@tscrim
Copy link
Collaborator

tscrim commented Oct 13, 2022

Reviewer: Travis Scrimshaw

@vbraun
Copy link
Member

vbraun commented Oct 17, 2022

Changed branch from u/mantepse/__floordiv___for_sparse_polynomials to 9577386

@vbraun vbraun closed this as completed in 9c07190 Oct 17, 2022
@mantepse mantepse mentioned this issue Nov 7, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: algebra p: major / 3 t: bug
Projects
None yet
Milestone
sage-9.8
Development

No branches or pull requests
3 participants
@vbraun @tscrim @mantepse