On Fri, May 30, 2014 at 09:28:30AM -0700, Mark Shoulson wrote:
>  > +def continued_fraction_rational(cf):
> 
>    continued_fraction_quadratic does reduce to continued_fraction_rational on
>    iterators with no repeating portion (no final element that is a list).
>    Should I try to combine the two into one function?

That make sense for me.  We don't have an additional
continued_fraction_periodic-like function for Rational's.

>    What would be a good
>    name for it?

I don't know.

>    Most of the time people will just be wanting to reduce a
>    normal terminating continued fraction back into its simple p/q form, so
>    the quadratic name could be confusing, but it's confusing in the other
>    direction without it.

We can't include in the SymPy every one-liner, sorry.