Closed ericbuckley closed 3 weeks ago
@cbrinson-rise8 I've been doing some testing on the new schema this week, https://github.com/CDCgov/RecordLinker/issues/62. Its looking good so far, performance is ever so slightly better, we're addressing some edge cases that the old code doesn't (eg last name O'Reilly or O'Connor), and its going to be way easier to make some of the changes NBS is asking for down the road.
At this point, I'd say don't worry about making changes to recordlinker.linkage
, just focus on the changes needed in recordlinker.linking
. The former packages is likely to be deleted soon.
Summary
The link.py and mpi.py modules expect a Dict for the algo_config parameter, update these to be an instance of the Algorithm class in models.py. We may also need to rewrite how the simple_mpi.py and simple_link.py use the algo_config, but only if those modules have been committed to main by the time this issue is worked on.
Acceptance Criteria
link_record_against_mpi
functionDependencies
15