#32275 closed defect (fixed)

Fix test failure with giac 1.7 after #32002

Reported by: Antonio Rojas Owned by:
Priority: major Milestone: sage-9.5
Component: symbolics Keywords:
Cc: Merged in:
Authors: Antonio Rojas Reviewers: Markus Wageringel
Report Upstream: N/A Work issues:
Branch: dd75684 (Commits, GitHub, GitLab) Commit: dd7568482c8dfdad57b8b52a5f337c8eb20f9ca2
Dependencies: Stopgaps:

Status badges

Description (last modified by Antonio Rojas)

This example is supposed to test a function that can't be integrated, but giac 1.7 does actually integrate it (in terms of algebraic numbers), which defeats its purpose. Replace it with a function that giac 1.7 can't integrate.

This wasn't failing before #32002 because maxima partially integrates it, so the maxima answer was returned instead.

As a side note: sage can't parse the giac 1.7 answer for the old integral, but that's a different problem.

Change History (5)

comment:1 Changed 14 months ago by Antonio Rojas

Branch: u/arojas/fix_test_failure_with_giac_1_7_after__32002

comment:2 Changed 14 months ago by Antonio Rojas

Authors: Antonio Rojas
Commit: dd7568482c8dfdad57b8b52a5f337c8eb20f9ca2
Component: PLEASE CHANGEsymbolics
Description: modified (diff)
Status: newneeds_review
Type: PLEASE CHANGEdefect

New commits:

dd75684Replace example with one that giac 1.7 can't integrate

comment:3 Changed 14 months ago by Markus Wageringel

Reviewers: Markus Wageringel
Status: needs_reviewpositive_review

LGTM.

comment:4 Changed 13 months ago by Matthias Köppe

Milestone: sage-9.4sage-9.5

comment:5 Changed 13 months ago by Volker Braun

Branch: u/arojas/fix_test_failure_with_giac_1_7_after__32002dd7568482c8dfdad57b8b52a5f337c8eb20f9ca2
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.