#2702 plotting option adapt_depth causes an error

None
closed
Yasuaki Honda
5
2014-03-17
2014-03-15
Yasuaki Honda
No

plot2d (w^2 * quad_qags (1/(s - w), s, 1, 5) [1], [w, -5, -1], [adapt_depth, 0]); causes an error.

Discussion

  • Yasuaki Honda
    Yasuaki Honda
    2014-03-15

    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1 +1 @@
    -plot2d (w^2 * quad_qags (1/(s - w), s, 1, 5) [1], [w, -5, -1], [adapt_depth, 0]);
    +plot2d (w^2 * quad_qags (1/(s - w), s, 1, 5) [1], [w, -5, -1], [adapt_depth, 0]); causes an error.
    
     
  • Yasuaki Honda
    Yasuaki Honda
    2014-03-15

    • labels: --> plotting, adapt_depth
     
  • Raymond Toy
    Raymond Toy
    2014-03-15

    Two issues here.

    First, a bug in quad_qags causes an error here. You currently need to use 'quad_qags here.

    Second, I think adapt_depth must be greater than 0.

     
  • Yasuaki Honda
    Yasuaki Honda
    2014-03-16

    • assigned_to: Yasuaki Honda
     
  • Yasuaki Honda
    Yasuaki Honda
    2014-03-16

    Raymond san wrote:
    "Second, I think adapt_depth must be greater than 0."

    In the past (like 5.30.0), 0 was accepted as the value of adapt_depth and 0 meant 'no splitting', according to the source code.
    As the latest plot.lisp in git explicitly checks if the value of adapt_depth is a natural number, I think I should update the test cases in rtest_plot.mac.

    I have updated rtest_plot.mac. Commit is [e567d9].

     

    Related

    Commit: [e567d9]


    Last edit: Yasuaki Honda 2014-03-16
  • Yasuaki Honda
    Yasuaki Honda
    2014-03-17

    • status: open --> closed