Skip to content

Let Mathematica free integration work with definite integrals #14764

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

Closed
kcrisman opened this issue Jun 18, 2013 · 21 comments
Closed

Let Mathematica free integration work with definite integrals #14764

kcrisman opened this issue Jun 18, 2013 · 21 comments

Comments

@kcrisman
Copy link
Member

See this ask.sagemath question. Apparently

sage: integrate(x^2,x,algorithm='mathematica_free')
1/3*x^3
sage: integrate(x^2,x,0,1,algorithm='mathematica_free')
1/3*x^3

so that would be useful to fix. Since it currently returns an indefinite integral when we ask for a definite one, I'll classify this as a defect.

Component: calculus

Author: Emmanuel Charpentier

Branch: 18385a7

Reviewer: Travis Scrimshaw, Frédéric Chapoton

Issue created by migration from https://trac.sagemath.org/ticket/14764

@kcrisman kcrisman added this to the sage-5.11 milestone Jun 18, 2013
@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@EmmanuelCharpentier
Copy link
Mannequin

EmmanuelCharpentier mannequin commented Jun 9, 2020

comment:5

This seems fixed in recent Sage versions. IN 9.2.beta0:

sage: integrate(x^2,x,0,1,algorithm='mathematica_free')
1/3
sage: integrate(sin(x),x,0,pi,algorithm='mathematica_free')
2
sage: integrate(sqrt(x),x,0,1,algorithm='mathematica_free')
2/3
sage: integrate(sqrt(x),x)
2/3*x^(3/2)

@EmmanuelCharpentier EmmanuelCharpentier mannequin removed this from the sage-6.4 milestone Jun 9, 2020
@kcrisman
Copy link
Member Author

kcrisman commented Jun 9, 2020

comment:6

Needs doctest if these aren't already tested (which they may be).

@EmmanuelCharpentier
Copy link
Mannequin

EmmanuelCharpentier mannequin commented Jun 9, 2020

comment:7

Replying to @kcrisman:

Needs doctest if these aren't already tested (which they may be).

Right. Putting this to needs_work.

@EmmanuelCharpentier
Copy link
Mannequin

EmmanuelCharpentier mannequin commented Jun 9, 2020

@EmmanuelCharpentier
Copy link
Mannequin

EmmanuelCharpentier mannequin commented Jun 9, 2020

Author: Emmanuel Charpentier

@EmmanuelCharpentier
Copy link
Mannequin

EmmanuelCharpentier mannequin commented Jun 9, 2020

@EmmanuelCharpentier
Copy link
Mannequin

EmmanuelCharpentier mannequin commented Jun 9, 2020

comment:10

Added relevant doctests to src/sage/symbolics/integratin/external.py, which pass for me.

==>needs_review

@EmmanuelCharpentier EmmanuelCharpentier mannequin added this to the sage-9.2 milestone Jun 9, 2020
@tscrim
Copy link
Collaborator

tscrim commented Jun 10, 2020

New commits:

ced138eDoctests for definite integral using mathematica_free.

@tscrim
Copy link
Collaborator

tscrim commented Jun 10, 2020

@tscrim
Copy link
Collaborator

tscrim commented Jun 10, 2020

Commit: ced138e

@tscrim
Copy link
Collaborator

tscrim commented Jun 10, 2020

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Jun 10, 2020

comment:12

Some fixes are needed:

-    ::
 
-    Check that :trac:`14764` is resolved:
+    Check that :trac:`14764` is resolved::

@EmmanuelCharpentier
Copy link
Mannequin

EmmanuelCharpentier mannequin commented Jun 10, 2020

comment:13

Aaaarghhh!!! Sphinx is exceedingly prone to punctuation errors.

I'm on it.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 10, 2020

Changed commit from ced138e to 18385a7

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 10, 2020

Branch pushed to git repo; I updated commit sha1. New commits:

18385a7Punctiation fic for doctests for definite integral using mathematica_free.

@EmmanuelCharpentier
Copy link
Mannequin

EmmanuelCharpentier mannequin commented Jun 10, 2020

comment:15

Done. needs_review

@fchapoton
Copy link
Contributor

Changed reviewer from Travis Scrimshaw to Travis Scrimshaw, Frédéric Chapoton

@fchapoton
Copy link
Contributor

comment:16

allons-y !

@vbraun
Copy link
Member

vbraun commented Jun 21, 2020

@seblabbe
Copy link
Contributor

Changed commit from 18385a7 to none

@seblabbe
Copy link
Contributor

comment:18

I think it was fixed in #25501

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

No branches or pull requests

7 participants