Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Ticket #954: ndimage.distance_transform_edt #203

Merged
merged 2 commits into from May 8, 2012

Conversation

Projects
None yet
2 participants
Contributor

jjhelmus commented May 4, 2012

One line correction to ndimage.distance_transform_edt which fixed segmentation fault upon passing False as mentioned in Ticket 954.

This is my first pull request so please be kind if I have made a mistake.

Owner

stefanv commented May 4, 2012

@jjhelmus Is there already a unit test to catch this problem? If not, we should add one.

Contributor

jjhelmus commented May 4, 2012

There is not a unit test yet. I'll add one when I'm back in the office (Monday).

Contributor

jjhelmus commented May 7, 2012

Unit test added, passes with updates, Segmentation fault without.

stefanv added a commit that referenced this pull request May 8, 2012

Merge pull request #203 from jjhelmus/ticket-954-fix
BUG: Fix segfault in distance_transform_edt (ticket 954).

@stefanv stefanv merged commit 005cdcf into scipy:master May 8, 2012

Owner

stefanv commented May 8, 2012

Good catch, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment