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

Untested and dead code in daubechies_scaling.hpp #1101

Open
jzmaddock opened this issue Feb 22, 2024 · 3 comments
Open

Untested and dead code in daubechies_scaling.hpp #1101

jzmaddock opened this issue Feb 22, 2024 · 3 comments

Comments

@jzmaddock
Copy link
Collaborator

Probably one for @NAThompson.

In daubechies_scaling.hpp

We have one line uncovered here:

which appears to be because the constructor is never called in our tests with anything except the default value for grid_refinements. We should have at least one absolute error (-2) value, plus one other positive value. But I'm not sure which tests and invariants hold in these cases?

Also the types detail::matched_holder and detail::linear_interpolation appear not to be used by anything, nor mentioned in the docs?

@jzmaddock
Copy link
Collaborator Author

Sorry, one other, there are no tests for double_prime() other than the boundary cases. Again I'm not sure how to validate the result?

@jzmaddock
Copy link
Collaborator Author

And one more, daubechies_wavelet.hpp is also untested for anything except default argument to daubechies_wavelet. The code for p==1 here:

is also untested.

@NAThompson
Copy link
Collaborator

@jzmaddock : Yup, you're right. Will try to get to this next week.

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