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

References added for watershed_ift algorithm #4496

Closed
wants to merge 5 commits into from

Conversation

nmoya
Copy link
Contributor

@nmoya nmoya commented Feb 5, 2015

Adding the original reference to the watershed transform implemented using the Image Foresting Transform algorithm.

@@ -1346,7 +1346,7 @@ def _hist(vals):

def watershed_ift(input, markers, structure=None, output=None):
"""
Apply watershed from markers using an iterative forest transform algorithm.
Apply watershed from markers using image foresting transform algorithm [1].
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should be [1]_ instead of [1].

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. I did a new commit to add the underscore.

@rgommers rgommers added Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org scipy.ndimage labels Feb 6, 2015
@rgommers
Copy link
Member

rgommers commented Feb 6, 2015

Adding this reference looks useful. There are a few minor style issues with this PR (first line too long, formatting of reference) that I fixed here: rgommers@04ee4028af3

I also squashed the commits and edited the commit message to be a more standard format. If that looks OK to you I'll merge that one.

@nmoya
Copy link
Contributor Author

nmoya commented Feb 6, 2015

@rgommers That looks great! I'm sorry I missed the standard. It looks great now.

rgommers added a commit that referenced this pull request Feb 8, 2015
@rgommers
Copy link
Member

rgommers commented Feb 8, 2015

@nmoya no worries, they're all minor things. Merged in commit 588559d. Thanks for the contribution!

@rgommers rgommers closed this Feb 8, 2015
@rgommers rgommers added this to the 0.16.0 milestone Feb 8, 2015
@nmoya nmoya deleted the watershed_ift_comments branch February 9, 2015 13:48
@nmoya nmoya restored the watershed_ift_comments branch February 9, 2015 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org scipy.ndimage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants