-
Notifications
You must be signed in to change notification settings - Fork 11
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] Overflow in mesh constructor #38
Comments
What's the status of this? |
I have not yet started debugging this issue |
Great can you de-assign yourself @ConduitDan and reassign when you're actively working on it. It may be something I jump in on (it's a minor error really...) |
So i think what we need to do is add range count checking to the functions in mesh builder. |
I'll start with a printed error with a return nil |
Agreed @ConduitDan I think this needs 0.6 and error raising... |
We've got error raising now! I'm assigning to me! |
Woohoo error raising! |
Revisiting this |
Thanks so much @ancaandrei for originally raising this issue. It's taken a while to get to—custom errors arrived in 0.6—but PR #287 has now fixed the issue and a test is in place. The fix will be forthcoming in v0.6.2. Thankyou again for your contribution! |
Describe the bug
Error 'IndxBnds': Index out of bounds.
in AreaMesh at line 82
from global at line 9
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Error should not be there or the error should say that the mesh has an upper bound.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: