Opened 6 years ago

Last modified 15 months ago

#20466 needs_work defect

Can't multiply continued fractions

Reported by: kcrisman Owned by:
Priority: major Milestone: sage-wishlist
Component: number theory Keywords: continued_fraction
Cc: vdelecroix, slelievre, tmonteil Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

There has been some reimplementation on here, so maybe this is no longer true, but ​this ask.sagemath post anyway points out 

sage: x = continued_fraction(pi)
sage: y = 3*x
TypeError: unsupported operand parent(s) for '*': 'Integer Ring' and '<class 'sage.rings.continued_fraction.ContinuedFraction_real'>'

Change History (8)

comment:1 Changed 6 years ago by slelievre

  • Cc vdelecroix slelievre tmonteil added
  • Keywords continued_fraction added

comment:2 Changed 6 years ago by vdelecroix

See #19120

comment:3 Changed 15 months ago by chapoton

  • Milestone changed from sage-7.2 to sage-duplicate/invalid/wontfix
  • Status changed from new to needs_review

maybe we can close as duplicate ?

comment:4 Changed 15 months ago by vdelecroix

Why would you? #19120 does not implement the full gosper algorithm for (a xy + b x + cy + d) / (e xy + f x + g y + h) but only (ax + b) / (cx + d).

comment:5 Changed 15 months ago by vdelecroix

Though that could work for 3*x.

comment:6 Changed 15 months ago by chapoton

indeed. Y a un algo pour multiplier deux fractions continues quelconques ?

comment:7 Changed 15 months ago by chapoton

  • Milestone changed from sage-duplicate/invalid/wontfix to sage-wishlist
  • Status changed from needs_review to needs_work
Note: See TracTickets for help on using tickets.