Appropriately handle race unknown

[bamader]

Description

This PR removes the feature_iter yield for the RACE field whenever an incoming record has a value for that field of UNKNOWN or ASKED_UNKNOWN. This ensures that, downstream, we don't perform fuzzy string comparisons against known race values and UNKNOWN and thereby award some log odds points where none should be.

Related Issues

#193

Additional Notes

n/a
<--------------------- REMOVE THE LINES BELOW BEFORE MERGING --------------------->

Checklist

Please review and complete the following checklist before submitting your pull request:

• I have ensured that the pull request is of a manageable size, allowing it to be reviewed within a single session.
• I have reviewed my changes to ensure they are clear, concise, and well-documented.
• I have updated the documentation, if applicable.
• I have added or updated test cases to cover my changes, if applicable.
• I have minimized the number of reviewers to include only those essential for the review.

Checklist for Reviewers

Please review and complete the following checklist during the review process:

• The code follows best practices and conventions.
• The changes implement the desired functionality or fix the reported issue.
• The tests cover the new changes and pass successfully.
• Any potential edge cases or error scenarios have been considered.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.65%. Comparing base (4d0ccfa) to head (0582e82).
Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #196   +/-   ##
=======================================
  Coverage   97.65%   97.65%           
=======================================
  Files          32       32           
  Lines        1576     1576           
=======================================
  Hits         1539     1539           
  Misses         37       37           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ericbuckley]

@bamader would you mind adding another test or two for this, the existing test doesn't account for ASKED_UNKNOWN. It also be good to assert that the other races are still being used appropriately.

[bamader]

@ericbuckley No problem, some test cases added to test for other races as well as ASKED_UNKNOWN

[ericbuckley]

LGTM