sympy/sympy — pull request #1357 by Krastanov (2012-06-16 13:46:09+00:00)
Title: Revert "Compute numerical roots if domain=RR (#1868)"

This is related to duplication of code and to code inconsistent with its docstrings in the poly module. See also:
http://code.google.com/p/sympy/issues/detail?id=3294
http://code.google.com/p/sympy/issues/detail?id=3295

This is not to be merged. However I hope that someone will explain to me what is the idea behind all the functions with slightly duplicated functionality (`nroots`, `roots(domain=RR)` before this revert and `RootOf.evalf`) None of them is using what is already implemented in the others.

Please check carefully the commented out failing tests. (Those that are not commented out are just the same tests, slightly changed in formating due to the revert).

The commit message:

```
This reverts commit f638fcfb46e760c22f6a664a1485b7f33fa56247.

Conflicts:

sympy/polys/polyroots.py
sympy/polys/tests/test_polyroots.py

The behavior of `roots` was not consistent with what was writtend in the
docstring. `roots` is documented as returning only radicals, however it was
returning also aproximante `float` solutions. `nroots`, `RootOf` and
`Poly.all_roots` already provide the functionality that was added here.

The behavior was changed to the old one, consistent with the
docstring and the docstring itself was updated to point out the standart ways
to get non-radical roots.
```