Skip to content
Commit 11f16976 authored by Matt Wala's avatar Matt Wala
Browse files

LineTaylorLocalExpansion: Reinstate the old derivative taking behavior 

when using sympy, so code generation doesn't blow up.

Also add a test the uses LineTaylorLocalExpansion.
parent 4e9d1eb1
Pipeline #6749 failed with stage
in 22 minutes and 35 seconds
Hide whitespace changes
Inline Side-by-side
  • Matt Wala @mattwala ·
    Author Maintainer

    The test isn't as good as I had hoped:

    • It's really slow under sympy
    • It still passes under the old version of the code, unless (I expect) the order gets really high

    So what is the right way to test this? Should I just test directly that the LineTaylorLocalExpansion derivatives don't blow up? @inducer, thoughts?

  • Matt Wala @mattwala ·
    Author Maintainer

    I added a regression test in d3f99ed0. I think there's no reason not to keep this test though.

0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment