Opened 4 years ago

Closed 4 years ago

#16941 closed enhancement (fixed)

Add a hold parameter for symbolic integrals

Reported by: tmonteil Owned by:
Priority: major Milestone: sage-6.4
Component: symbolics Keywords:
Cc: Merged in:
Authors: Thierry Monteil Reviewers: Ralf Stephan
Report Upstream: N/A Work issues:
Branch: 721dc23 (Commits) Commit: 721dc23353e720f8bea963792c4a8808e15ee8f4
Dependencies: Stopgaps:

Description

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.

Change History (6)

comment:1 Changed 4 years ago by tmonteil

  • Branch set to u/tmonteil/add_a_hold_parameter_for_symbolic_integrals

comment:2 Changed 4 years ago by tmonteil

  • Authors set to Thierry Monteil
  • Commit set to 721dc23353e720f8bea963792c4a8808e15ee8f4
  • Status changed from new to needs_review

New commits:

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

comment:3 Changed 4 years ago by rws

  • Status changed from needs_review to positive_review

Really needed. Simple patch and buildbot is happy.

comment:4 Changed 4 years ago by vbraun

  • Status changed from positive_review to needs_work

Reviewer name

comment:5 Changed 4 years ago by rws

  • Reviewers set to Ralf Stephan
  • Status changed from needs_work to positive_review

comment:6 Changed 4 years ago by vbraun

  • Branch changed from u/tmonteil/add_a_hold_parameter_for_symbolic_integrals to 721dc23353e720f8bea963792c4a8808e15ee8f4
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.