-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
108-resize #125
108-resize #125
Conversation
@wyli Tests will be added to PR soon. |
@wyli @ericspod By default, skimage.transform.rotate sets preserve_range to False. What happens is
outputs [[[4.82469967e-18 4.93311988e-18 5.04154010e-18 5.14996032e-18] [[3.08997619e-18 3.19839641e-18 3.30681663e-18 3.41523684e-18] [[1.35525272e-18 1.46367293e-18 1.57209315e-18 1.68051337e-18] I think this is misleading behavior. By default preserve_range should be True so
outputs [[[44. 45. 46. 47.] [[28. 29. 30. 31.] [[12. 13. 14. 15.] |
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.
I think preserve_range
should be by default True, that seems like expected behaviour.
0561e7c
to
d0a5244
Compare
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.
Added comments inline.
Thanks.
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.
Added comments inline.
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.
Looks good to me.
Fixes #108 .
Description
Resize an image (Spatial Scale). Tests coming soon.
Status
Work in progress
Types of changes