search

kappazeta / km_predict

S2 full image prediction
Apache License 2.0
20 stars 9 forks source link

Km predict s3 test #31

Closed asrithaweo closed 1 month ago

indrek-sunter commented 1 month ago

To me it looks good. Perhaps

  1. some of the comments in docker/km_predict/copernicus_s3_utils.py look a bit odd,
  2. additional indentation might be good for clarity in docker/km_predict/km_s3.sh.
thierryweo commented 1 month ago

@indrek-sunter Hello we are working on a fallback mechanism to download missing sentinel-2 tiles from the Copernicus repo. Is this something you would be interested in having merged back upstream? In this case we make fix the comments above and reopen a new PR?

Greetings