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

WaveNet head check is not working properly #107

Closed
mikeoliphant opened this issue Aug 1, 2024 · 1 comment · Fixed by #108
Closed

WaveNet head check is not working properly #107

mikeoliphant opened this issue Aug 1, 2024 · 1 comment · Fixed by #108

Comments

@mikeoliphant
Copy link
Contributor

The WaveNet check to make sure there is no "head" element in the json has a couple of problems that are currently canceling each other out under normal circumstances.

This line:

const bool with_head = config["head"] == NULL;

has two problems. First, the logic is reversed, and it looks like it should incorrectly be setting "with_head" to "true" when it is null.

This is currently being compensated for by the fact that having:

"head": null

in the json actually doesn't make config["head"] null - it is rather a json object that references null.

So, everything currently works ok, but the check isn't really doing what it is supposed to do.

@mikeoliphant
Copy link
Contributor Author

It turns out that the existing code is actually causing problems on some systems:

mikeoliphant/neural-amp-modeler-lv2#69

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

Successfully merging a pull request may close this issue.

1 participant