Ticket #13845 (closed defect: fixed)

Opened 5 months ago

Last modified 5 months ago

BooleanPolynomialRing.random_element chokes on terms parameter of type rational

Reported by: malb Owned by: AlexGhitza
Priority: major Milestone: sage-5.6
Component: basic arithmetic Keywords:
Cc: AlexanderDreyer, PolyBoRi Work issues:
Report Upstream: None of the above - read trac for reasoning. Reviewers: Alexander Dreyer
Authors: Martin Albrecht Merged in: sage-5.6.beta2
Dependencies: Stopgaps:

Description

This shouldn't produce an error: 

sage: n = 10
sage: B = BooleanPolynomialRing(n, 'x')
sage: B.random_element(terms=(n/2)**2))

Attachments

trac_13845.patch Download (1.4 KB) - added by malb 5 months ago.

Change History

Changed 5 months ago by malb

comment:1 Changed 5 months ago by malb

  • Cc AlexanderDreyer, PolyBoRi added
  • Status changed from new to needs_review
  • Authors set to Martin Albrecht

comment:2 Changed 5 months ago by AlexanderDreyer

  • Status changed from needs_review to positive_review
  • Reviewers set to Alexander Dreyer

I can give positive review for that patch.

comment:3 Changed 5 months ago by jdemeyer

  • Status changed from positive_review to closed
  • Resolution set to fixed
  • Merged in set to sage-5.6.beta2
Note: See TracTickets for help on using tickets.