Skip to content

fix: add resolution to python down/up sampling operators#1199

Open
jdroenner wants to merge 5 commits into
mainfrom
python_sampling_resolution
Open

fix: add resolution to python down/up sampling operators#1199
jdroenner wants to merge 5 commits into
mainfrom
python_sampling_resolution

Conversation

@jdroenner

Copy link
Copy Markdown
Member

Here is a brief summary of what I did:

  • Added serialization of more Interpolation and Downsampling parameters to Python
  • Added error tracing to WebSocket handlers to catch errors on the server Side (don't only send them)

@coveralls

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 27023555140

Coverage increased (+0.1%) to 87.318%

Details

  • Coverage increased (+0.1%) from the base build.
  • Patch coverage: 17 uncovered changes across 3 files (13 of 30 lines covered, 43.33%).
  • 6 coverage regressions across 3 files.

Uncovered Changes

File Changed Covered %
python/geoengine/workflow_builder/operators.py 22 12 54.55%
geoengine/services/src/api/handlers/workflows.rs 6 0 0.0%
python/geoengine/types.py 1 0 0.0%
Total (4 files) 30 13 43.33%

Coverage Regressions

6 previously-covered lines in 3 files lost coverage.

File Lines Losing Coverage Coverage
geoengine/services/src/datasets/external/edr.rs 3 89.09%
python/geoengine/workflow_builder/operators.py 2 77.37%
geoengine/services/src/datasets/external/aruna/mock_grpc_server.rs 1 83.12%

Coverage Stats

Coverage Status
Relevant Lines: 134299
Covered Lines: 117267
Line Coverage: 87.32%
Coverage Strength: 482078.82 hits per line

💛 - Coveralls

@jdroenner jdroenner marked this pull request as ready for review June 9, 2026 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants