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

BF: Put back utils.length. #299

Merged
merged 5 commits into from Dec 24, 2013

Conversation

Projects
None yet
4 participants
@arokem
Member

arokem commented Dec 24, 2013

No description provided.

def length(streamlines, affine=None):
"""

This comment has been minimized.

@MrBago

MrBago Dec 24, 2013

Contributor

Do you want to add a one liner here?

arokem added some commits Dec 24, 2013

@@ -2,15 +2,17 @@
"""This module is the python part of dipy.tracking.utils.py, it was split
into another file to avoid circular imports."""
import itertools

This comment has been minimized.

@MrBago

MrBago Dec 24, 2013

Contributor

You can drop this import now that you're import map from six

This comment has been minimized.

@arokem

arokem Dec 24, 2013

Member

Yep. Just did that :-)

On Mon, Dec 23, 2013 at 7:37 PM, MrBago notifications@github.com wrote:

In dipy/tracking/_utils.py:

@@ -2,15 +2,17 @@
"""This module is the python part of dipy.tracking.utils.py, it was split
into another file to avoid circular imports."""

+import itertools

You can drop this import now that you're import map from six


Reply to this email directly or view it on GitHubhttps://github.com//pull/299/files#r8538829
.

Returns
-------
terator object which then computes the length of each

This comment has been minimized.

@samuelstjean

samuelstjean Dec 24, 2013

Contributor

Iterator object, not terator

Garyfallidis added a commit that referenced this pull request Dec 24, 2013

Merge pull request #299 from arokem/utils-length
BF: Put back utils.length.

@Garyfallidis Garyfallidis merged commit a87a07e into nipy:master Dec 24, 2013

1 check passed

default The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment