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

Add an explicit check for the correct number of material indices in smooth_\varepsilon #27

Open
gvnwst opened this issue May 8, 2024 · 0 comments

Comments

@gvnwst
Copy link
Collaborator

gvnwst commented May 8, 2024

If a geometry generator is switched out and the material indices array is not correctly updated, you can either have a silent error (where potentially the wrong material is applied) or a pretty esoteric error like

ERROR: LoadError: BoundsError: attempt to access NTuple{4, Int64} at index [5]

but vaguely inside smooth_\varepsilon somewhere.

Adding an explicit check or assertion that the number of provided material indices matches the number of geometric objects would make debugging both these errors much faster.

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

1 participant