Skip to content

WIP: Add split_multindex #52

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

Closed
wants to merge 1 commit into from
Closed

WIP: Add split_multindex #52

wants to merge 1 commit into from

Conversation

jakirkham
Copy link
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
Owner Author

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

@jakirkham
Copy link
Owner Author

Maybe split_indices is a little clearer name.

@jakirkham
Copy link
Owner Author

jakirkham commented Feb 17, 2017

Might also be worthwhile to have an inverse like join_indices.

@jakirkham jakirkham mentioned this pull request Feb 17, 2017
@jakirkham
Copy link
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