-
Notifications
You must be signed in to change notification settings - Fork 61
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
Fix diff roi #267
Fix diff roi #267
Conversation
…rom correlation.py
This function will provide the indices, number of pixels and pixel indices when a circular roi is divided into pies
@tacaswell I think I have to close the PR #248 Divide the circle : find the indices, number of pixels, pixel indices. I fixed this new PR, I fixed all the functions in this , the test are passing but there is a problem with the coverage,(missed 7 lines) |
Suggested name changes:
|
Scikit-image has fast functions based on published CS literature for drawing rings: http://scikit-image.org/docs/dev/api/skimage.draw.html#skimage.draw.circle |
|
||
return mesh.reshape(detector_size), roi_inds, num_pixels, pixel_list | ||
return mesh.reshape(detector_size), roi_labels, indices |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only return the first bit, not the second two.
What is this PR? |
Fixed the functions in the diff_roi_choice.py after changes in the correlation.py