search

swisstopo / topo-satromo

Erdbeobachtungs-Satellitendaten fürs Trockenheitsmonitoring (SATROMO)
BSD 3-Clause "New" or "Revised" License
14 stars 3 forks source link

asset CloudProbability not uint8 #73

Closed davidoesch closed 7 months ago

davidoesch commented 7 months ago

Describe the bug with CS+ cloud score is not uint8 but double with v0.0.1

To Reproduce Steps to reproduce the behavior: projects/satromo-prod/assets/col/S2_SR_HARMONIZED_SWISS/S2-L2A_mosaic_2023-08-20T103631_bands-10m

9 | cloudProbability | double -- | -- | --

Expected behavior as in projects/satromo-prod/assets/col/S2_SR_HARMONIZED_SWISS/S2-L2A_mosaic_2024-03-14T102811_bands-10m

9 | cloudProbability | unsigned int8 -- | -- | --

Screenshots expected result image

OS, python version -v0.0.1

Additional context

cs+ needs to be casted to uint8 before exporting

needs a fix for

davidoesch commented 7 months ago

fixed eg in projects/satromo-int/assets/COL_S2_SR_HARMONIZED_SWISS/S2-L2A_mosaic_2020-07-11T103629_bands-10m

Tschoun commented 7 months ago

Fixed the data type for S2_TOA here: f37d71b42317150b50e9458a1b0f202c39dde4e3 Will be merged to the main branch with the next merge. Java Script fixed in prod environment.

davidoesch commented 7 months ago

fixed existing data on PRD with https://github.com/swisstopo/topo-satromo/blob/dev/main_functions/util_castassetbands.py