nasaharvest / crop-mask

End-to-end workflow for generating high resolution cropland maps
Apache License 2.0
95 stars 28 forks source link

Cropland: Tanzania North 2019 #202

Closed ivanzvonkov closed 1 year ago

ivanzvonkov commented 1 year ago

Key tasks:

adebowaledaniel commented 1 year ago

Completed! View in EE here

ivanzvonkov commented 1 year ago

Reposting slack message here: Looks pretty good upon visual inspection

Looks like many of those missing values you got were in the ocean I also see a decent amount of ERA5 looking grid artifacts, so next step would be to retrain the model with the skip_ERA5 argument and rerun predictions. This time the data is all already exported so no need to wait for Earth Engine.

adebowaledaniel commented 1 year ago

Pull request for the skip_ERA5 model (https://github.com/nasaharvest/crop-mask/pull/205).

adebowaledaniel commented 1 year ago

Tanzania North 2019 (skip ERA5) crop mask view here.

ivanzvonkov commented 1 year ago

Tanzania North 2019 (skip ERA5) crop mask view here.

The script has an issue: Line 1: image2 is not defined

Could you also add both results to the same script so we can compare? @adebowaledaniel

adebowaledaniel commented 1 year ago

Sorry about that; here is the updated link.

With both results, you mean with and without ERA5? @ivanzvonkov

ivanzvonkov commented 1 year ago

Yes both with and without ERA5

adebowaledaniel commented 1 year ago

You can compare both results here

adebowaledaniel commented 1 year ago

@ivanzvonkov The updated comparison view is here. I will mosaic the multiple rasters generated for each model if needed in another issue.

ivanzvonkov commented 1 year ago

Excellent! Looks like there are some more fields found in the North East now and a lot fewer grid artifacts. I didn't realize that the f1-score is also higher when skipping ERA5! 🤔 https://github.com/nasaharvest/crop-mask/blob/494d1d1d7e21dff64e2fcafe2e12887f6d680e7d/data/models.json#L87

@gabrieltseng @hannah-rae interesting result here!

cnakalembe commented 1 year ago

Reviewed and ready to merge with Tanzania South

adebowaledaniel commented 1 year ago

Excellent! Looks like there are some more fields found in the North East now and a lot fewer grid artifacts. I didn't realize that the f1-score is also higher when skipping ERA5! 🤔

https://github.com/nasaharvest/crop-mask/blob/494d1d1d7e21dff64e2fcafe2e12887f6d680e7d/data/models.json#L87

@gabrieltseng @hannah-rae interesting result here!

@ivanzvonkov, what is the following line of action with this? Is there a conclusion on the performance of with/without ERA5 based on other models trained? Moving forward, which model versions should be used for the final merge of both Tanzania North and South?