-
Notifications
You must be signed in to change notification settings - Fork 136
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
CascadeStudio parameter sanity checking #37
Comments
@udif As a workaround until something more robust is in place, you can simply log the computed radius (and anything else you want to debug) to the console using |
Yes, I am making use of console.log() and alert(). |
Unfortunately, any errors that happen within the OpenCascade kernel are undebuggable due to the way it was compiled. :( There's a PR for an Embind recompilation of OpenCascade (which should have better error reporting), but I haven't had a ton of time lately to work out all of the kinks in there, so it's been sitting in PR Queue hell since early October... |
What I implied by this issue was to add individual sanity checks in OpenCascade for each command before it is issued to the OpenCascade kernel library. In any case, the major issue I'm having is #38. If this was solved, there would be less need for issues such as this one (although it is pretty useful by itself). |
It seems that CascadeStudio is not doing any sanity checking on parameters. As a result, a call such as:
Will result in:
When this is a part of a larger script, and the parameters are due to long calculations, it is very hard to debug the scripts.
This is an example only, there are other unchecked calls.
The text was updated successfully, but these errors were encountered: