Opened 7 years ago

Closed 7 years ago

#11947 closed enhancement (fixed)

Implement cardinality for polynomial quotient rings

Reported by: jdemeyer Owned by: AlexGhitza
Priority: minor Milestone: sage-4.8
Component: basic arithmetic Keywords:
Cc: Merged in: sage-4.8.alpha5
Authors: Jeroen Demeyer Reviewers: John Cremona
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by cremona)

Looks good and applies fine to 4.8.alpha4. All long tests in sage/rings pass.

Attachments (1)

11947.patch (1.6 KB) - added by jdemeyer 7 years ago.

Download all attachments as: .zip

Change History (6)

Changed 7 years ago by jdemeyer

comment:1 Changed 7 years ago by jdemeyer

  • Status changed from new to needs_review

comment:2 Changed 7 years ago by jdemeyer

  • Authors set to Jeroen Demeyer

comment:3 Changed 7 years ago by jdemeyer

  • Milestone sage-4.7.3 deleted

Milestone sage-4.7.3 deleted

comment:4 Changed 7 years ago by cremona

  • Description modified (diff)
  • Milestone set to sage-4.8
  • Reviewers set to John Cremona
  • Status changed from needs_review to positive_review

comment:5 Changed 7 years ago by jdemeyer

  • Merged in set to sage-4.8.alpha5
  • Resolution set to fixed
  • Status changed from positive_review to closed

Thanks for the review!

Note: See TracTickets for help on using tickets.