Skip to content

WIP: Add split_multindex - #52

Closed
jakirkham wants to merge 1 commit into
masterfrom
add_split_multindex
Closed

jakirkham wants to merge 1 commit into
masterfrom
add_split_multindex

Conversation

@jakirkham

Copy link
Copy Markdown
Owner

Adds a utility function, split_multindex, for splitting up slices with multiple indices into a list of slices each with a single index. This is handy when working with libraries that accept slices that have a single index, but balk when multiple indices are provided. In these cases, one can split up the multiple indices into multiple slices, which can then be iterated over. Construction of the total result is up to user.

Adds a utility function, `split_multindex`, for splitting up slices with
multiple indices into a list of slices each with a single index. This is
handy when working with libraries that accept slices that have a single
index, but balk when multiple indices are provided. In these cases, one
can split up the multiple indices into multiple slices, which can then
be iterated over. Construction of the total result is up to user.
@jakirkham jakirkham changed the title Add split_multindex WIP: Add split_multindex Feb 13, 2017
@jakirkham

Copy link
Copy Markdown
Owner Author

Probably could use some more tests, but the basic functionality is there.

@jakirkham

Copy link
Copy Markdown
Owner Author

Maybe split_indices is a little clearer name.

@jakirkham

jakirkham commented Feb 17, 2017 •

Copy link
Copy Markdown
Owner Author

Might also be worthwhile to have an inverse like join_indices.

@jakirkham jakirkham mentioned this pull request Feb 17, 2017
@jakirkham

Copy link
Copy Markdown
Owner Author

Replacing with PR ( #53 ).

@jakirkham
jakirkham deleted the add_split_multindex branch March 13, 2017 17:21
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