surgeo icon indicating copy to clipboard operation
surgeo copied to clipboard

First Name / Last Name / Geocode Mixins

Open theonaunheim opened this issue 4 years ago • 5 comments

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.

theonaunheim avatar Jan 25 '21 16:01 theonaunheim

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.

TheCleric avatar Jan 25 '21 16:01 TheCleric

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.

theonaunheim avatar Jan 26 '21 03:01 theonaunheim

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.

TheCleric avatar Jan 26 '21 03:01 TheCleric

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

theonaunheim avatar Feb 01 '21 16:02 theonaunheim

@theonaunheim absolutely!

TheCleric avatar Feb 01 '21 16:02 TheCleric