Changes between Version 31 and Version 40 of Ticket #12455


Ignore:
Timestamp:
06/20/13 01:43:31 (6 years ago)
Author:
burcin
Comment:

trac_12455-airy_review.patch fixes a couple of problems:

  • airy_{a,b}i_general should evaluate to airy_{a,b}i_simple, airy_{a,b}i_prime, etc. depending on the first parameter
  • differentiating airy_{a,b}i_general in the first parameter is not allowed
  • _evalf_ should not raise errors for unrecognized algorithm argument

This is still needs work because we use indirect doctests everywhere. There is no reason to go through the airy_{a,b}i wrapper functions. We should call the symbolic functions directly in the doctests. There are also a lot of code paths that are not tested. I changed the fractional order behavior, but didn't need to change any doctests.

It would be great if someone else can fix/add doctests. I will move on to something else now.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #12455

    • Property Status changed from needs_review to needs_work
    • Property Authors changed from Oscar Gerardo Lazo Arjona, Benjamin Jones, Eviatar Bach to Oscar Gerardo Lazo Arjona, Benjamin Jones, D. S. McNeil, Eviatar Bach
    • Property Reviewers changed from Eviatar Bach, Karl-Dieter Crisman to Eviatar Bach, Karl-Dieter Crisman, Burcin Erocal
  • Ticket #12455 – Description

    v31 v40  
    3838----
    3939
    40 Apply [attachment:trac_12455-newstyle-airy-rebase.patch] and [attachment:trac_12455-newstyle-airy2-rebase.patch].
     40Apply
     41
     42 * [attachment:trac_12455-newstyle-airy-rebase.patch]
     43 * [attachment:trac_12455-newstyle-airy2-rebase.patch]
     44 * [attachment: trac_12455-airy_review.patch]