Comment #15 on issue 1456 by smi...@gmail.com: use pyflakes to identify simple bugs in sympy and fix them
http://code.google.com/p/sympy/issues/detail?id=1456

There are still many undefined variables which is bad: this means that an error is waiting to happen in code that is not being tested.

e.g.

sympy/benchmarks/bench_meijerint.py:7: W802 undefined name 'inverse_fourier_transform' sympy/benchmarks/bench_meijerint.py:8: W802 undefined name 'inverse_laplace_transform' sympy/benchmarks/bench_meijerint.py:9: W802 undefined name 'inverse_mellin_transform'
sympy/categories/diagram_drawing.py:2082: W802 undefined name 'obj'
sympy/core/tests/test_function.py:231: W802 undefined name 'oo'

And there is a lot of cruft in terms of unused variables which is less critical but could be a good place for beginners to start.

e.g.


sympy/abc.py:1: W402 'Symbol' imported but unused
    it's in an exec

sympy/galgebra/GA.py:2249: W806 redefinition of function 'div' from line 2238 sympy/galgebra/tests/test_GA.py:83: W806 local variable 'Bsq' is assigned to but never used sympy/galgebra/tests/test_GA.py:116: W806 local variable 'e' is assigned to but never used

I don't know if this issue will ever close. These things crop up like weeds unless the pep8/pyflakes/flake8 tools are not used regularly.

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

Reply via email to