8000 Additional households fixes based on testing by dehall · Pull Request #58 · mitre/data-owner-tools · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Additional households fixes based on testing #58

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

Merged
merged 7 commits into from
Apr 26, 2023
Merged

Conversation

dehall
Copy link
Collaborator
@dehall dehall commented Apr 20, 2023
  • Ensure the "written_to_file" flag is read properly
  • Tweak match thresholds to reduce tumbleweeds, in particular in apartments
  • Fix for referencing a None object
  • Adds a new --exact_addresses option
    • This uses a faster but lower quality method for address comparison. By default the inference process will split up addresses into street, number, suffix, etc, and use family name and phone number, but this increases memory & runtime. Enabling this feature causes the process to use the entire address as a single string for comparisons, and only the address is considered. If addresses have not been standardized/validated, this setting will increase false negatives (records not being included in households where they should be). This is primarily intended to serve as a baseline for testing the software, but might be used in practice as a last resort.
  • Prints some basic household summary stats at the end
  • Also adds an extra print out at the end of linkid_to_patid of the name of the filename that was written

@dehall dehall changed the title Additional fixes based on testing Additional households fixes based on testing Apr 20, 2023
@dehall dehall merged commit 6ec0195 into master Apr 26, 2023
@dehall dehall deleted the further_household_fixes branch April 26, 2023 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0