theonaunheim / surgeo

Open Source Proxy Demographic module written in Python
MIT License
32 stars 16 forks source link

Make "ALL OTHER NAMES" and forename equivalent available #18

Open theonaunheim opened 3 years ago

theonaunheim commented 3 years ago

Currently returns NaN because it is normalized to "ALLOTHERNAMES".

theonaunheim commented 3 years ago

pranav_thing.pdf

Kludgy workaround to impute values instead of calculating with NaNs.

jrm87 commented 3 years ago

Thank you! This is very helpful. I also noticed that when I pass series that have NaNs in some parts, but all series have the same length, get_probabilities() returns a pandas that is larger than any of the initial series. I think this might correct for that as well, but I'll confirm.