Comment #2 on issue 1945 by smichr: Fix as_numer_denom() docstring and add Expr to the Sphinx docs
http://code.google.com/p/sympy/issues/detail?id=1945

That's a comment directed toward the idea of processing Eqs as has been discussed recently, but I added that in the code so that anyone being tempted to make the change would see that note. We don't have a good mechanism for including such comments that document possible changes to the code that don't work at present. There
are other such examples that I have found useful, e.g. in mul.py

        # Unfortunately, this isn't smart enough to consider combining into
        # exponents that might already be adds, so things like:
        #  z - y    y
# x * x will be left alone. This is because checking every possible
        # combination can slow things down.

Also there is the whole issue of where to include the docstrings for methods:
- is there even a way to see the docstring of a method?
- when a method is subclassed should the main class instance have all the docstrings or should they be in the subclassed methods? e.g. "def as_numer_denom" appears in
exponential, numbers, powers, mul, expr, basic, and add.

--
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