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

lazy view version of imrotate/imresize #119

Open
johnnychen94 opened this issue May 19, 2021 · 0 comments
Open

lazy view version of imrotate/imresize #119

johnnychen94 opened this issue May 19, 2021 · 0 comments

Comments

@johnnychen94
Copy link
Member

johnnychen94 commented May 19, 2021

imrotate is a high-level interface to warp. WarpedView is a lazy version of warp, and it makes sense to also have a lazy version of imrotate built on top of WarpedView.

When tweaking the performance, the lazy version is usually a good choice because:

  • it does not allocate new memory for the entire array
  • it does not compute the result for all pixels (only happens when you fetch it via getindex)

The lazy version is not always the best choice because if you call getindex repeatedly, the computation happens repeatedly.

Edit:

Similarily, we can support lazy imresize.

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

No branches or pull requests

1 participant