Menu

#2318 Patch for hann dgrid3d kernel

None
closed-accepted
nobody
None
2020-12-07
2020-09-24
No

Dear all,

At present hann kernel function for dgrid3d is implemented as
w(d) = 0.5 * (1 − cos(2*π*d)) * 1(d<1),
which seems to wrong as it takes 0 at d=0.
The function should be
w(d) = 0.5 * (1 + cos(π*d)) * 1(d<1).
Please find the attached patch.

Best

1 Attachments

Discussion

  • Ethan Merritt

    Ethan Merritt - 2020-09-24

    Second error in the dgrid3d code reported recently, 12 years after the code was added. See Bug #2277. It must not have been tested much initially and not used much since then.
    Thanks.

     
  • Ethan Merritt

    Ethan Merritt - 2020-09-24
    • status: open --> pending-accepted
    • Group: Version 5 -->
    • Priority: 5 -->
     
  • Ethan Merritt

    Ethan Merritt - 2020-09-24

    Applied for 5.4 and 5.5

     
  • Ethan Merritt

    Ethan Merritt - 2020-12-07
    • Status: pending-accepted --> closed-accepted
     

Log in to post a comment.

Auth0 Logo