Skip to content
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

integrate: Correctly handle manual, meijerg, risch flags #14475

Merged
merged 1 commit into from Mar 12, 2018
Merged

integrate: Correctly handle manual, meijerg, risch flags #14475

merged 1 commit into from Mar 12, 2018

Commits on Mar 11, 2018

  1. integrate: Correctly handle manual, meijerg, risch flags

    Only one of manual, meijerg, risch can be True, since True means "use
    only this method" (per [the documentation of
    `integrate`](http://docs.sympy.org/latest/modules/integrals/integrals.html#api-reference)).
    This was not checked.
    ```
    >>> integrate(1/x, x, risch=True, manual=True)
    log(x)
    ```
    Also, the option manual=True did not actually prevent meijerg integration
    ```
    >>> integrate(exp(x**2), x, manual=True)
    sqrt(pi)*erfi(x)/2
    ```
    Both are corrected now: ValueError is raised if more than one True is
    given, and whenever one is True, the others are set to False.
    normalhuman committed Mar 11, 2018
    Configuration menu
    Copy the full SHA
    b257cae View commit details
    Browse the repository at this point in the history