Opened 9 years ago

Last modified 22 months ago

#12731 closed defect

Disable or raise warning with abs_integrate as default — at Version 9

Reported by: roed Owned by: burcin
Priority: critical Milestone: sage-8.9
Component: calculus Keywords: abs_integrate
Cc: nbruin, mjo, kcrisman, pbruin, rws, paulmasson, mforets Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by kcrisman)

Maxima's abs_integrate contributed package gives us lots of nice new integrals, but also really breaks a lot of stuff, apparently. Add to this list as you see fit:

  • #13733 (fixed but note unwanted interactions)
  • #11590 sgn() function (not reported upstream?)
  • #14591 cosh integral incorrect because of abs_integrate
  • #16643 integrate() infinite loop
  • #17183 another abs_integrate error
  • #17468 log(abs(sin())) problem
  • #17511 abs(sin(x)) and abs(cos(x)) are flat-out wrong

Old:

Can someone more knowledgeable about using maxima for integration determine what the right warning to show is and when to show it? See #12691 for what a stopgap is.

Change History (9)

comment:1 Changed 9 years ago by kcrisman

I'm not really sure that this is appropriate for a stopgap. I mean, unless we put in a catch for the specific form like integrate(x * sgn(x^2 - 1/4), x, -1, 0), we would have to put in a stopgap for the entire integration, which is ridiculous.

Unless we checked for the signum function with nonlinear operands in it sometime... but that's more than I have time to parse now.

comment:2 Changed 9 years ago by jdemeyer

  • Priority changed from blocker to major

comment:3 Changed 8 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:4 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:5 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:6 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:7 Changed 6 years ago by kcrisman

  • Cc pbruin rws added
  • Description modified (diff)
  • Summary changed from Stopgap for #11590 to Disable abs_integrate as default

comment:8 Changed 6 years ago by kcrisman

  • Summary changed from Disable abs_integrate as default to Disable or raise warning with abs_integrate as default

Naturally, if it becomes unnecessary to do so, we can just close this as wontfix. This is the best equivalent to a stopgap I can come up with - or possibly we can raise a warning every time abs_integrate is used, though this won't stop the hangs and other weird interactions it causes.

comment:9 Changed 6 years ago by kcrisman

  • Description modified (diff)
Note: See TracTickets for help on using tickets.