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.

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

to_scl_dilation_mask: only one band allowed #270

Open
soxofaan opened this issue Mar 11, 2024 · 0 comments
Open

to_scl_dilation_mask: only one band allowed #270

soxofaan opened this issue Mar 11, 2024 · 0 comments
Assignees
@soxofaan

Comments

@soxofaan
Copy link
Member

to_scl_dilation_mask currently specifies

openeo-python-driver/openeo_driver/specs/openeo-processes/experimental/to_scl_dilation_mask.json

Line 13 in faeb345

"description": "The source data cube containing only the Sentinel-2 SCL band.",

But user can accidentally provide input cube with multiple bands (e.g. B02, B03, SCL), which gives weird results

this should trigger an error howerver

(FYI: this was initial issue in Open-EO/openeo-geopyspark-driver#715 )
@soxofaan soxofaan self-assigned this Mar 11, 2024
soxofaan added a commit that referenced this issue Mar 11, 2024
@soxofaan
Issue #270 to_scl_dilation_mask: specify defaults in spec
b12ac98
soxofaan added a commit that referenced this issue Mar 11, 2024
@soxofaan
Issue #270 to_scl_dilation_mask: use defaults from spec
8a8ebf4
soxofaan added a commit that referenced this issue Mar 11, 2024
@soxofaan
Issue #270 to_scl_dilation_mask: define as DriverDataCube method
ef1e297 
makes things easier to track
soxofaan added a commit that referenced this issue Mar 11, 2024
@soxofaan
Issue #270 to_scl_dilation_mask: raise error when input has multiple …
cdaa88c 
…bands
soxofaan added a commit that referenced this issue Mar 12, 2024
@soxofaan
fixup! Issue #270 to_scl_dilation_mask: raise error when input has mu…
2c604f2 
…ltiple bands
soxofaan added a commit that referenced this issue Mar 26, 2024
@soxofaan
Issue #270 to_scl_dilation_mask: specify defaults in spec
a62aac3
soxofaan added a commit that referenced this issue Mar 26, 2024
@soxofaan
Issue #270 to_scl_dilation_mask: use defaults from spec
04b2fb7
soxofaan added a commit that referenced this issue Mar 26, 2024
@soxofaan
Issue #270 to_scl_dilation_mask: define as DriverDataCube method
4b76b09 
makes things easier to track
soxofaan added a commit that referenced this issue Mar 26, 2024
@soxofaan
Issue #270 to_scl_dilation_mask: raise error when input has multiple …
6670bb5 
…bands
soxofaan added a commit that referenced this issue Mar 26, 2024
@soxofaan
Issue #270 to_scl_dilation_mask: raise error when input has multiple …
c0538e4 
…bands
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@soxofaan soxofaan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@soxofaan @JeroenVerstraelen