Updates:
        Cc: [email protected] [email protected]

Comment #38 on issue 1337 by asmeurer: Refactor find() and replace()
http://code.google.com/p/sympy/issues/detail?id=1337

I agree that the documentation needs to be better. It should be clear on the list of things in the replace docstring what they do, i.e., just

3.1. func -> func
     obj.replace(lambda expr: ..., lambda expr: …)

doesn't make it clear exactly what it does (in particular, it isn't clear that the first function is evaluated as a boolean). And the examples should be referenced in a way that makes it clearer what example corresponds to what combination.

I don't think all the combinations make sense. Like you can't just put a type in an arbitrary place (you can only put a type where a type was), and you can't put an expr where a type was (unless you had issue 2006). But you should be able to do func -> expr (in fact, the given example doctest for that would make more sense to do it that way).

find() needs a way better docstring, that actually explains what all the options do and has doctests.

Ronan and/or Mateusz, do you want to work on this? Otherwise, it will be postponed until after the release.

--
You received this message because you are subscribed to the Google Groups 
"sympy-issues" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sympy-issues?hl=en.

Reply via email to