Skip to content

array_contains is not supported in the OpenEOProcessScriptBuilder #392

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.

Sign up for GitHub

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

Jump to bottom
Open
JeroenVerstraelen opened this issue Mar 11, 2025 · 2 comments · May be fixed by #415
Open

array_contains is not supported in the OpenEOProcessScriptBuilder #392

JeroenVerstraelen opened this issue Mar 11, 2025 · 2 comments · May be fixed by #415
Assignees
@dsamaey @JeroenVerstraelen
Labels
bug Something isn't working

Comments

@JeroenVerstraelen
Copy link
Contributor

cc @VictorVerhaert
@JeroenVerstraelen JeroenVerstraelen self-assigned this Mar 11, 2025
@VictorVerhaert VictorVerhaert added the bug Something isn't working label Mar 11, 2025
@VictorVerhaert
Copy link

added the bug label as currently the process is promoted as supported

@jdries
Copy link
Contributor

jdries commented Apr 7, 2025

The implementation of array_find is probably the best to copy paste from:

openeo-geotrellis-extensions/openeo-geotrellis/src/main/scala/org/openeo/geotrellis/OpenEOProcessScriptBuilder.scala

Line 1205 in 5233412

case "array_find" => arrayFind(arguments)

For unit tests: we can now write tests based on sample process graphs, best to try and use that for this one.

dsamaey pushed a commit that referenced this issue Apr 10, 2025
Issue #392 added support for "array_contains" in the OpenEOProcessScr…
49c9450 
…iptBuilder
@dsamaey dsamaey linked a pull request Apr 10, 2025 that will close this issue
Issue #392 added support for "array_contains" in the OpenEOProcessScr… #415
Open
dsamaey pushed a commit that referenced this issue Apr 10, 2025
Issue #392 fixed compilation issue
3a67b6c
dsamaey pushed a commit that referenced this issue Apr 23, 2025
Issue #392 array_contains support (PR remarks)
86b99e2
dsamaey pushed a commit that referenced this issue Apr 24, 2025
Issue #392 array_contains support (fixed tests)
8f87ed6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dsamaey dsamaey

@JeroenVerstraelen JeroenVerstraelen

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
4 participants
@dsamaey @JeroenVerstraelen @jdries @VictorVerhaert