Skip to content

Commit

Permalink
src/sage/symbolic/integration/external.py: update a comment
Browse files Browse the repository at this point in the history
A comment in this file about failing doctests is still partially true
(the doctests fail if you remove the workaround), but it mentions a
"fixme" that does not exist. We delete the nonsense part.
  • Loading branch information
orlitzky committed Dec 4, 2024
1 parent 56d5c3a commit f384ef5
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions src/sage/symbolic/integration/external.py
Original file line number Diff line number Diff line change
Expand Up @@ -256,10 +256,9 @@ def libgiac_integrator(expression, v, a=None, b=None):
return expression.integrate(v, a, b, hold=True)

from sage.libs.giac.giac import Pygen
# We call Pygen on first argument because otherwise some expressions
# involving derivatives result in doctest failures in interfaces/sympy.py
# -- related to the fixme note in sage.libs.giac.giac.GiacFunction.__call__
# regarding conversion of lists.
# We call Pygen on first argument because otherwise some
# expressions involving derivatives result in doctest failures in
# sage/interfaces/sympy.py
if a is None:
result = libgiac.integrate(Pygen(expression), v)
else:
Expand Down

0 comments on commit f384ef5

Please sign in to comment.