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] RS_MapAlgebra for performing map algebra operations using simple expressions #970

Merged
merged 1 commit into from
Aug 16, 2023

Conversation

Kontinuation
Copy link
Member

@Kontinuation Kontinuation commented Aug 15, 2023

Did you read the Contributor Guide?

Is this PR related to a JIRA ticket?

What changes were proposed in this PR?

Add a new function RS_MapAlgebra for performing map algebra function using Jiffle scripts. This will make implementing map algebra operations much easier than using array-based functions.

We've also fixed several bugs in DeepCopiedRenderedImage:

  1. The sample model of DeepCopiedRenderedImage became inconsistent with the wrapped raster after deserialization;
  2. Serialize GC_NODATA property, which is a special property used by GeoTools to identify nodata values of raster images.

How was this patch tested?

Added unit tests for RS_MapAlgebra.

Did this PR include necessary documentation updates?

@Kontinuation Kontinuation force-pushed the map-algebra-single-raster branch 7 times, most recently from edeff28 to 49ff50e Compare August 15, 2023 10:10
@Kontinuation Kontinuation marked this pull request as ready for review August 15, 2023 10:20
@jiayuasu jiayuasu added this to the sedona-1.5.0 milestone Aug 16, 2023
@jiayuasu jiayuasu merged commit 0794529 into apache:master Aug 16, 2023
39 checks passed
@Kontinuation Kontinuation deleted the map-algebra-single-raster branch August 23, 2023 15:01
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