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

Add a hold parameter for symbolic integrals #16941

Closed
sagetrac-tmonteil mannequin opened this issue Sep 6, 2014 · 8 comments
Closed

Add a hold parameter for symbolic integrals #16941

sagetrac-tmonteil mannequin opened this issue Sep 6, 2014 · 8 comments

Comments

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented Sep 6, 2014

As discussed in this ask question, there is no hold parameter for integration:

sage: integral(x^2, x, 0, 3, hold=True)
TypeError: integrate() got an unexpected keyword argument 'hold'

However definite_integral and indefinite_integral support this paramter:

sage: from sage.symbolic.integration.integral import definite_integral
sage: definite_integral(x^2, x, 0, 3, hold=True)
integrate(x^2, x, 0, 3)
sage: latex(_)
\int_{0}^{3} x^{2}\,{d x}

Since integral basically calls the previous functions, we should transfer this parameter.

Component: symbolics

Author: Thierry Monteil

Branch/Commit: 721dc23

Reviewer: Ralf Stephan

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

@sagetrac-tmonteil sagetrac-tmonteil mannequin added this to the sage-6.4 milestone Sep 6, 2014
@sagetrac-tmonteil
Copy link
Mannequin Author

sagetrac-tmonteil mannequin commented Sep 6, 2014

@sagetrac-tmonteil
Copy link
Mannequin Author

sagetrac-tmonteil mannequin commented Sep 6, 2014

New commits:

721dc23#16941 : Add a hold parameter for symbolic integrals.

@sagetrac-tmonteil
Copy link
Mannequin Author

sagetrac-tmonteil mannequin commented Sep 6, 2014

Author: Thierry Monteil

@sagetrac-tmonteil
Copy link
Mannequin Author

sagetrac-tmonteil mannequin commented Sep 6, 2014

Commit: 721dc23

@rwst
Copy link

rwst commented Sep 7, 2014

comment:3

Really needed. Simple patch and buildbot is happy.

@vbraun
Copy link
Member

vbraun commented Sep 7, 2014

comment:4

Reviewer name

@rwst
Copy link

rwst commented Sep 7, 2014

Reviewer: Ralf Stephan

@vbraun
Copy link
Member

vbraun commented Sep 8, 2014

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

2 participants