WorldCereal / presto-worldcereal

4 stars 0 forks source link

Seasonality check for specific regions #50

Closed cbutsko closed 5 months ago

cbutsko commented 7 months ago

Using a simple find_peaks function from scipy, existing training data for WorldCereal was analysed. 12-month timeseries of NDVI were obtained for each cropland sample. As a result, each sample received a corresponding number of detected vegetation peaks. Then, for each country and each n_peaks value, a random subset of 100 samples was selected (if available) and plotted.

Below is the list of countries where seasonality might require adjustment:

And here are charts for visual clarity: Image Image Image Image Image Image Image Image

More detailed findings can be found in issue 13

@MolettoLobos Can we check what seasons look like in the new version of crop calendar for these countries? It is also very likely that neighboring countries will have the same issue, however they are not included in this list as we don't have enough points there.

kvantricht commented 7 months ago

@cbutsko the Kazakhstan plot caught my attention because many of the curves are cut off. Any idea why? Is this 0 values you're now not showing?

cbutsko commented 7 months ago

@kvantricht yes, these are absent values. I suppose no clean imagery was available. The same situation can be seen in Latvia (one plot shows Oats recognized as crop, another one - those that are not recognized as crop) Image Image

cbutsko commented 6 months ago

@MolettoLobos Italo, any thoughts or updates regarding this issue?

MolettoLobos commented 6 months ago

Hi Christina, sorry for late reply. For the season retrieval how are you retrieving the data? for SOS and EOS are you using the AEZ? or the raster with cropcalendars? In case the usage is the AEZ? In those countries the behaviour seems to be shifted, and such places like Mexico contains different type of crops. We can arrange a meeting to see what happens and understand better the behaviour. I think these parameters can be fixed with LSP incorporation

kvantricht commented 6 months ago

@MolettoLobos @cbutsko does this issue need to remain open or do we create other ones for specific required corrections?

cbutsko commented 5 months ago

I believe we can close this one for now. The last thing we agreed on is that we need to monitor performance of new calendars on the countries listed in the description of this issue. Once this is ready, I think Italo @MolettoLobos can reopen and update this issue or make a new one with more specific corrections.