Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug in example / fnft_kdvv_example.c #79

Open
xmhk opened this issue Aug 3, 2023 · 1 comment
Open

bug in example / fnft_kdvv_example.c #79

xmhk opened this issue Aug 3, 2023 · 1 comment

Comments

@xmhk
Copy link

xmhk commented Aug 3, 2023

In: 0.5 release / master branch
when compiled, the kdvv c-example (examples/fnft_kdvv_example.c) fails:

FNFT Error: Invalid argument opts->grid_spacing.
 in kdvv_compute_boundstates(593)-0.5.0
FNFT Error: Subroutine failure.
 in fnft_kdvv_base(480)-0.5.0
FNFT Error: Subroutine failure.
 in fnft_kdvv(242)-0.5.0
An error occured!

I suppose this is due to using the default grid-spacing of 0.0.
The example works fine if the grid spacing is explicitly defined, e.g. as

fnft_kdvv_opts_t opts = fnft_kdvv_default_opts();
// define grid spacing 
opts.grid_spacing = 0.001;

Best, Christoph

@wahls
Copy link
Contributor

wahls commented Aug 4, 2023

Hi Christoph,

Many thanks for reporting. The grid_spacing default is intentionally set to zero, so that the user has to choose this (important parameter) him-/herself. There are other algorithms for the discrete vKdV spectrum that do not utilize a grid (not implemented yet), which is why I did not make it a main argument for the function.

I'll put this on the list of things to fix for 0.5.1. If you notice anything else, please let me know.

Best, Sander

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants