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

[SEDONA-361] Fix performance issues of map algebra functions and some band operators #1008

Merged
merged 1 commit into from
Sep 7, 2023

Conversation

Kontinuation
Copy link
Member

Did you read the Contributor Guide?

Is this PR related to a JIRA ticket?

What changes were proposed in this PR?

We've spotted several performance hits of map algebra functions:

  1. ImageUtils.createConstantImage is very slow, we'd better manually create a buffered image, which is way faster.
  2. The .getData() method on BufferedImage objects creates an unnecessary copy of the raster object. We've added a utility function to implement a fast path for BufferedImage.

How was this patch tested?

Passing existing tests

Did this PR include necessary documentation updates?

  • No, this PR does not affect any public API so no need to change the docs.

@jiayuasu jiayuasu added this to the sedona-1.5.0 milestone Sep 7, 2023
@jiayuasu jiayuasu merged commit bd331f0 into apache:master Sep 7, 2023
78 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants