Opened 6 years ago

Closed 4 years ago

#16699 closed defect (fixed)

degree of multivariate polynomial rings with weighted degree is inconsistent

Reported by: lftabera Owned by:
Priority: major Milestone: sage-6.4
Component: algebra Keywords: weighted degree
Cc: Merged in:
Authors: Luis Felipe Tabera Alonso Reviewers: Bruno Grenet
Report Upstream: N/A Work issues:
Branch: 6f72a78 (Commits) Commit: 6f72a788e5714f1d3c9fed7bec5ac113836d4019
Dependencies: Stopgaps:

Description

Depending on the type fo the ring (Singular vs polydict) the return of degree differs.

sage: R1.<x,y>=PolynomialRing(QQ,'x,y',order=TermOrder('wdeglex',(2,3)))
sage: (x+y).degree()
3
sage: R2.<u,v>=PolynomialRing(QQ['t'],'x,y',order=TermOrder('wdeglex',(2,3)))
sage: (u+v).degree()
1

Change History (6)

comment:1 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:2 Changed 5 years ago by lftabera

  • Authors set to Luis Felipe Tabera Alonso
  • Branch set to u/lftabera/ticket/16699
  • Commit set to 6f72a788e5714f1d3c9fed7bec5ac113836d4019
  • Status changed from new to needs_review

New commits:

6f72a78degree of polydict multivariate polynomial rings with weighted term order

comment:3 Changed 4 years ago by bruno

  • Status changed from needs_review to positive_review

This looks good to me, passes the tests!

Though some bot seems to be unhappy with this ticket, I have not been able to find a problem so that I turn it to positive_review.

comment:4 Changed 4 years ago by vbraun

  • Status changed from positive_review to needs_work

Reviewer name

comment:5 Changed 4 years ago by bruno

  • Reviewers set to Bruno Grenet
  • Status changed from needs_work to positive_review

comment:6 Changed 4 years ago by vbraun

  • Branch changed from u/lftabera/ticket/16699 to 6f72a788e5714f1d3c9fed7bec5ac113836d4019
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.