Skip to content

Argument check in EdgeIterator inner constructor#50

Open
vlc1 wants to merge 1 commit into
JuliaArrays:masterfrom
vlc1:vlc1/empty-edgeiterator
Open

Argument check in EdgeIterator inner constructor#50
vlc1 wants to merge 1 commit into
JuliaArrays:masterfrom
vlc1:vlc1/empty-edgeiterator

Conversation

@vlc1
Copy link
Copy Markdown

@vlc1 vlc1 commented May 10, 2024

The current inner constructor for EdgeIterator throws an error when the inner field is empty:

julia> inner = CartesianIndices((0,));

julia> outer = CartesianIndices((1,));

julia> EdgeIterator(outer, inner)
ERROR: DimensionMismatch: CartesianIndices((0,)) must be in the interior of CartesianIndices((1,))

Fix calls issubset instead.

empty inner and/or outer fields.
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 this pull request may close these issues.

1 participant