Closed rabdill closed 1 year ago
It seems a good idea. Perhaps drop that file in the Zenodo release?
I thought this would be beneficial behavior, to make sure we always have the most up to date geo info. But as you said it might be a problem when names of things change and no longer match up to our data, so we should probably version lock it somehow. I'll put it in the repo.
As for Zenodo, maybe you just want to refer to the version you're using?
The first step in the
processNaturalEarth
function in the compilation script currently fetches the most recent "ne_110m_admin_0_countries" Natural Earth data. This is probably as reliable a source as any, but it would still fail from changes to file names or organization. It looks like the file is small enough that it would be practical to store a copy of it and update later if there's a new release—would that be preferable? It couldn't hurt to stay updated, but it doesn't seem urgent enough that we need to pull straight from master every time. It looks like their license wouldn't preclude this.