New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make Gosper code use new dispersion algorithm #205

Merged
merged 3 commits into from Jan 31, 2016

Conversation

2 participants
@skirpichev
Copy link
Collaborator

skirpichev commented Jan 30, 2016

First commit - from sympy/sympy#2702

raoulb and others added some commits Dec 21, 2013

Check if alpha is Integer in dospersionset
The following code will not work for symbolic
integers in general (failing example is test_issue_8016).

Also, do some cleanup (don't call sympify, for example).

@skirpichev skirpichev force-pushed the gosper-disp branch from adf60b5 to da81e33 Jan 30, 2016

skirpichev added a commit that referenced this pull request Jan 31, 2016

Merge pull request #205 from skirpichev/gosper-disp
Make Gosper code use new dispersion algorithm

@skirpichev skirpichev merged commit 5cf1765 into master Jan 31, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.03%) to 92.64%
Details

@skirpichev skirpichev deleted the gosper-disp branch Jan 31, 2016

@skirpichev

This comment has been minimized.

Copy link
Collaborator

skirpichev commented Feb 2, 2016

BTW, I hope @raoulb - ok with inclusion of such not-yet-merged-to-sympy code here. I always will preserve your commit ownership and will add strict indication if your commit was modified in some way, except trivial conflict fixes.

@raoulb

This comment has been minimized.

Copy link
Contributor

raoulb commented Feb 2, 2016

@skirpichev skirpichev modified the milestone: 0.8.0 Sep 9, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment