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

support for weighted term orders in normal_basis #29625

Closed
mwageringel opened this issue Apr 30, 2020 · 7 comments
Closed

support for weighted term orders in normal_basis #29625

mwageringel opened this issue Apr 30, 2020 · 7 comments

Comments

@mwageringel
Copy link

As a follow-up to #29543, this ticket changes the normal_basis method of ideals to handle the case of weighted term orders.

With this change, the degree of the monomials in the normal basis is taken with respect to the weighted degree (which agrees with Sage's notion of degree).

sage: R.<x,y,z> = PolynomialRing(QQ, order=TermOrder('wdegrevlex', (1, 2, 3)))
sage: I = R.ideal(x*y^2 + x^5, z*y + x^3*y)
sage: I.normal_basis(degree=9)
[x^2*y^2*z, x^3*z^2, x*y*z^2, z^3]
sage: all(f.degree() == 9 for f in _)
True

This also came up in an Ask SageMath question.

The implementation uses the Singular function weightKB.

CC: @kwankyu @tscrim @heluani

Component: commutative algebra

Keywords: singular

Author: Markus Wageringel

Branch/Commit: f0604bd

Reviewer: Travis Scrimshaw

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

@mwageringel
Copy link
Author

Author: Markus Wageringel

@mwageringel
Copy link
Author

Commit: f0604bd

@mwageringel
Copy link
Author

Branch: u/gh-mwageringel/29625

@mwageringel
Copy link
Author

New commits:

f0604bd29625: support weighted term orders in normal_basis

@tscrim
Copy link
Collaborator

tscrim commented May 1, 2020

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented May 1, 2020

comment:2

LGTM.

@vbraun
Copy link
Member

vbraun commented May 4, 2020

Changed branch from u/gh-mwageringel/29625 to f0604bd

@vbraun vbraun closed this as completed in ec4fd4a May 4, 2020
@mkoeppe mkoeppe modified the milestones: sage-9.2, sage-9.1 May 4, 2020
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