Handle NAs in randomNames()'s gender argument about randomnames HOT 3 CLOSED

Thanks for seeing that. I've fixed that up. Let me know if there's anything else. Version 1.3-0.0 is the new version. Available on GitHub now and will post to CRAN shortly and hopefully available there in a day or two.

from randomnames.

Thanks, Damian, for looking into this. As far as I can tell, the non-NA genders are now populated correctly.

May I ask what behavior you've chosen for the cases where gender is NA? I can think of multiple options that seem sensible:

• Return NA in the name vector
• Random draw a name from the names tables (without considering gender)
• Random draw a gender from the non-missing elements of the gender vector

Possibly, the information on the chosen behavior could be included in the documentation of the function.

from randomnames.

Good idea (to include in documentation).

Current it just randomly samples a gender/ethnicity and replaces the NA with that. https://github.com/CenterForAssessment/randomNames/blob/master/R/randomNames.R#L61-L62

from randomnames.