Skip to content
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

Generate sequential IDs #124

Open
binnympaul opened this issue Jul 30, 2020 · 2 comments
Open

Generate sequential IDs #124

binnympaul opened this issue Jul 30, 2020 · 2 comments

Comments

@binnympaul
Copy link
Collaborator

Having long household IDs or geographic IDs can possible result in an error. Simple solution is to pre-process inputs to inlcude sequential IDs. Perhaps PopulationSim can generate sequential IDs on the fly.

@bettinardi
Copy link
Collaborator

My preferred solution would be to have populationsim tread HH ids as character strings. Person ids make sense as numbers, but HH ids are just names (like the Bettinardi household). There should be no reason that the software needs to treat them as characters - at least no reason I can think of.

Internal renaming can work, and is likely a cheaper (faster) option, but that feels like a solution that is more likely to create or mask errors. My strong preference would be to treat HH ids as names and not care how long the name is (within reason, maybe set the character string length at 255, or less).

@bettinardi
Copy link
Collaborator

My add on comment to make hh ids names is probably overkill. For Phase 9, perhaps there could be a low level effort to add sequential IDs generated on the fly under Phase 9.

@bettinardi bettinardi added this to the Phase 9 Priorities milestone Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants