mapme-initiative / mapme.biodiversity

Efficient analysis of spatial biodiversity datasets for global portfolios
https://mapme-initiative.github.io/mapme.biodiversity/
GNU General Public License v3.0
24 stars 7 forks source link

Add GSW indicators #234

Closed karpfen closed 4 months ago

karpfen commented 5 months ago

The global surface water data are available via their website. The aim is to include a selection of them in the package, namely

DoD:

Update 2024-02-19

karpfen commented 5 months ago

GSW indicators are finished and included in the mapme package now (see https://github.com/mapme-initiative/mapme.biodiversity/pull/235).

karpfen commented 5 months ago
goergen95 commented 5 months ago

Just realised that the resources are called global_surface_water_* whereas the indicators are called gsw_*. Would you mind applying the abbreviation? If we do it simply, we would have resources and indicators with the same name (which is not a problem from the technical standpoint, I guess) but maybe we also could think of a smarter way to name things :laughing: Its not just the GSW resource, but applies to others as well... You have any ideas?

goergen95 commented 5 months ago

Well, yeah, if we think of #232 it might actually be a problem in the future...

karpfen commented 5 months ago

I think I'll open a new issue for renaming things, because I feel that will be a bigger one and I have some ideas already.

goergen95 commented 5 months ago

@karpfen, the GSW resources currently lack proper documentation. Could you please provide them?

karpfen commented 4 months ago

@goergen95 sorry, I must have missed your comment, just saw it. I documented the individual indicators in their respective .calc* functions (e.g. https://github.com/mapme-initiative/mapme.biodiversity/blob/main/R/calc_gsw_occurrence.R#L3), is that what you mean?

goergen95 commented 4 months ago

Yep, but they do not show up for users, e.g. here. Please refer to other resources/indicators how to document them properly. There is also a section on that in the developers guide.