-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Inverse Radon transform: Expand functionality #683
base: main
Are you sure you want to change the base?
Conversation
@stefanv @emmanuelle Any chance of a review? |
skimage/transform/radon_transform.py
Outdated
@@ -156,6 +160,10 @@ def iradon(radon_image, theta=None, output_size=None, | |||
Assume the reconstructed image is zero outside the inscribed circle. | |||
Also changes the default output_size to match the behaviour of | |||
``radon`` called with ``circle=True``. | |||
projection_shifts : 1D array, dtype=float |
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.
optional
I put two comments. Is it possible to illustrate this option in the gallery? |
@sciunto Thanks for looking into this. About the gallery, I think we should be careful with including too many features in what is now intended to be an introductory example. The correct place for examples of the use of these features is in a longer, tutorial-like introduction to tomography in scikit-image, I think. And I don't want to write that yet, as the tomography part of scikit-image still feels somewhat incomplete. |
oki :) |
@stefanv Any suggestions for who I can ask for a review for this one? |
This PR does two things:
projection_shifts
argument toiradon
, with identical functionality as foriradon_sart
Both changes are intended to be backwards compatible.
From my side, this is ready for merge.