Menu ▾ ▴

#2702 plotting option adapt_depth causes an error

None
closed
Yasuaki Honda
plotting (138) adapt_depth (1)
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
     

Log in to post a comment.