theonaunheim / surgeo

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

First Name / Last Name / Geocode Mixins #15

Open theonaunheim opened 3 years ago

theonaunheim commented 3 years ago

People are more likely to have forename and surnames together than they are to have surname and geo data together.

Since the data is already there, examine the possibility of updating surname probability with forname.

TheCleric commented 3 years ago

Would it make sense to have some sort of generic system to combine models without having logic in each that's a bit repetitive?

This would also allow moving functions that are somewhat model specific (like _get_prob_race_given_first_name()) from the base_model level down to their parent model level.

theonaunheim commented 3 years ago

Sounds solid. If you want to get this in prior to your article, let me know. Otherwise I'll probably put this in with the rewrite that will coincide with the 2020 census data.

TheCleric commented 3 years ago

Sounds solid. If you want to get this in prior to your article, let me know. Otherwise I'll probably put this in with the rewrite that will coincide with the 2020 census data.

No rush for me.

theonaunheim commented 3 years ago

@TheCleric , can you hit me up on LinkedIn or let me know here when you write your article? I'd like to read it.

TheCleric commented 3 years ago

@theonaunheim absolutely!