Opened 14 months ago

Closed 13 months ago

## #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: |

### Description (last modified by )

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

Branch: | → u/arojas/fix_test_failure_with_giac_1_7_after__32002 |
---|

### comment:2 Changed 14 months ago by

Authors: | → Antonio Rojas |
---|---|

Commit: | → dd7568482c8dfdad57b8b52a5f337c8eb20f9ca2 |

Component: | PLEASE CHANGE → symbolics |

Description: | modified (diff) |

Status: | new → needs_review |

Type: | PLEASE CHANGE → defect |

### comment:3 Changed 14 months ago by

Reviewers: | → Markus Wageringel |
---|---|

Status: | needs_review → positive_review |

LGTM.

### comment:4 Changed 13 months ago by

Milestone: | sage-9.4 → sage-9.5 |
---|

### comment:5 Changed 13 months ago by

Branch: | u/arojas/fix_test_failure_with_giac_1_7_after__32002 → dd7568482c8dfdad57b8b52a5f337c8eb20f9ca2 |
---|---|

Resolution: | → fixed |

Status: | positive_review → closed |

**Note:**See TracTickets for help on using tickets.

New commits:

`Replace example with one that giac 1.7 can't integrate`