-
Notifications
You must be signed in to change notification settings - Fork 20
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
Authenticated internal use #5
Comments
And or use this package to add DHS data. |
Hey, I was sending someone who wanted to get all the data used for the malaria maps to this package and noticed the DHS coordinates were missing and then saw this issue :) The following gets you very close to what you may want. I've started it in a fork, but there were a couple of Most the function documentation is the same as that for Let me know what you think/any ideas on the odd dhs_ids Ta, OJ
|
Thank you very much for getting in touch! I think having this as a separate function like you've got here is the best design. Given that it's Christmas it might take me a little while to look at this fully. But I'm sure we'll merge it in. Thanks again! |
If we want this package to replace many lines of code in our internal work, we need it to get the DHS data (and other private data) via authenticated access.
Ideally, we want to keep this as hidden from external users as possible. So for example, we don't want
d <- getPR('ZAF', username = 'tim', password = 'mystrongpassword')
because the docs on CRAN will just have to say "this is not for you". Which is not good software documentation.
The text was updated successfully, but these errors were encountered: