[sympy] WIP - trigsimp modifications (#1690)

0 views
Skip to first unread message

Christopher Smith

unread,
Dec 7, 2012, 5:01:06 AM12/7/12
to sympy/sympy

You can merge this Pull Request by running:

  git pull https://github.com/smichr/sympy trig

Or view, comment on, or merge it at:

  https://github.com/sympy/sympy/pull/1690

Commit Summary

  • trigsimp modifications

File Changes

  • M sympy/simplify/simplify.py (159)
  • M sympy/simplify/tests/test_simplify.py (25)

Patch Links


Reply to this email directly or view it on GitHub.

Christopher Smith

unread,
Dec 7, 2012, 5:02:37 AM12/7/12
to sympy/sympy

@asmeurer, maybe I can bring back some of the features of the old trigsimp branch. These are some pretty basic modifications with a few new patterns.

Julien Rioux

unread,
Dec 7, 2012, 5:41:43 PM12/7/12
to sympy/sympy

SymPy Bot Summary: :eight_spoked_asterisk: Passed after merging smichr/trig (ea33d48) into master (361240a).
:eight_spoked_asterisk:PyPy 2.0.0-beta-1; 2.7.3-final-42: pass
:eight_spoked_asterisk:Python 2.7.2-final-0: pass
:eight_spoked_asterisk:Python 3.2.1-final-0: pass
:eight_spoked_asterisk:Sphinx 1.1.3: pass

Aaron Meurer

unread,
Dec 7, 2012, 6:15:11 PM12/7/12
to sympy/sympy

SymPy Bot Summary: :red_circle: Failed after merging smichr/trig (ea33d48) into master (9c41978).
@smichr: Please fix the test failures.
:red_circle:Python 2.5.0-final-0: fail
:red_circle:Python 2.6.6-final-0: fail
:red_circle:Python 2.7.2-final-0: fail
:eight_spoked_asterisk:Python 2.6.8-final-0: pass
:eight_spoked_asterisk:Python 2.7.3-final-0: pass


:eight_spoked_asterisk:PyPy 2.0.0-beta-1; 2.7.3-final-42: pass

:eight_spoked_asterisk:Python 3.2.2-final-0: pass
:eight_spoked_asterisk:Python 3.3.0-final-0: pass
:eight_spoked_asterisk:Python 3.2.3-final-0: pass
:eight_spoked_asterisk:Python 3.3.0-final-0: pass
:eight_spoked_asterisk:Python 3.3.0-final-0: pass
:red_circle:**Sphinx 1.1.3:** fail

Aaron Meurer

unread,
Dec 7, 2012, 11:52:25 PM12/7/12
to sympy/sympy

Could you write a more descriptive commit message?

Christopher Smith

unread,
Dec 8, 2012, 6:30:49 AM12/8/12
to sympy/sympy

Before I remove the WIP there will be something more descriptive.

Christopher Smith

unread,
Dec 9, 2012, 5:45:05 AM12/9/12
to sympy/sympy

Closed #1690.

Christopher Smith

unread,
Dec 9, 2012, 5:45:05 AM12/9/12
to sympy/sympy

To get back to where #772 is going to take more time than I can give right now so I'll close this.

Christopher Smith

unread,
Dec 11, 2012, 4:56:49 AM12/11/12
to sympy/sympy

Reopened #1690.

Christopher Smith

unread,
Dec 11, 2012, 4:57:21 AM12/11/12
to sympy/sympy

I gave it another try today...let's see how it goes with tests.

Christopher Smith

unread,
Dec 11, 2012, 5:35:37 AM12/11/12
to sympy/sympy

issues addressed can be found by searching for PR 772 in the issues (#772).

Christopher Smith

unread,
Dec 11, 2012, 11:49:54 PM12/11/12
to sympy/sympy

Don't commit -- tests need to be added for the patterns.

Aaron Meurer

unread,
Dec 15, 2012, 11:26:44 PM12/15/12
to sympy/sympy

Even if you can't get to it, I would still leave it open. There's a chance someone else will pick up the work. At the very least, if you close it, open an issue about it, so that the work is not lost completely.

Reply all
Reply to author
Forward
0 new messages