-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Separate out data into a separate package #4
Comments
I had previously wondered about calling it rcountries, but happy to consider other options. I had thought it would be good to include the country synonyms joining stuff with modernised code. I did have a reproducible workflow from the natural earth data to the sp objects, it's here : https://github.com/AndySouth/rworldmapSetup/blob/master/saveMapPolygons.r I recently broke that convention for a user request about Western Sahara & Somaliland. What has stopped me from updating to the more recent Natural Earth boundaries is that the field names changed which could break a bunch of rworldmap examples, and I don't really have time to fix all that. Suggestions appreciated. |
I quite like If the field names have changed, then maybe it's reasonable to make a clean break and create a data only package, and then later figure out how to update rworldmap to use the new data? I can help out with some of the steps in the data processing. I think it should be possible to automate the downloads, and hence maybe include some more of the natural earth datasets. If the package is data only, CRAN is usually not so strict about the size limitations. I think there's probably also a better solution to the encoding problem, and I'm happy to look into it. I'd also be tempted to remove as much of the fix-up code as possible, instead relying on the quality of Natural Earth data to improve over time. (And encourage people to report problems upstream) |
I think it's also worth thinking through what to do with the attached data - is it better to keep everything, or just keep (say) the ISO3 code, and then provide another data frame that makes it easier to link to other naming schemes. |
That all sounds good. Particularly the clean break and making naturalearth findable. Also as you say there are a whole load of fields from the shapefiles that aren't needed and could be left out. For the name how about rnaturalearth ? Partly because that word doesn't exist yet so would be easy to track it. |
Sure, sounds good to me. |
Plus would it be good to see if the rOpenSci folks would like to be involved ? I've been wanting an excuse to get involved. What's best to do, get it started and go from there ? |
@karthik how do we get started? |
@hadley Sounds great. If you start a repo on either of your accounts and drop me a note, we can go from there. Once the package is completed, we have a quick review and onboarding process to get it into the ropensci suite. |
So, would this also move the map data out of |
Thanks @karthik @hadley @hrbrmstr. I just created the repo at https://github.com/AndySouth/rnaturalearth. Do add youself in as you want. I'll start to outline tomorrow what I think should be in. Is the readme a good place to do that or are there any other recommended practices for planning ? |
See : ropensci/rnaturalearth#2 |
Maybe rworldmap data? Or naturalearth?
It could also contain the code used to do processing from the raw data to the appropriate sp objects. If you could point me to that code, I'd be happy to start the ball rolling
The text was updated successfully, but these errors were encountered: