Make map conversions more efficient alternative

[jarq6c]

This is an alternative to #188 . This PR removes the conversion step when checking contingency tables, resulting in much faster computations for categorical metrics.

Additions

• None

Removals

• convert_mapping_values

Changes

• contingency tables are converted directly to pandas.Series of the appropriate type.

Testing

1. Removed test for convert_mapping_values

Notes

Todos

Checklist

• PR has an informative and human-readable title
• PR is well outlined and documented. See #12 for an example
• Changes are limited to a single goal (no scope creep)
• Code can be automatically merged (no conflicts)
• Code follows project standards (see CONTRIBUTING.md)
• Passes all existing automated tests
• Any change in functionality is tested
• New functions are documented (with a description, list of inputs, and expected output) using numpy docstring formatting
• Placeholder code is flagged / future todos are captured in comments
• Reviewers requested with the Reviewers tool ➡️

[jarq6c]

@groutr hopefully you don't mind reviewing this PR? :)

[jarq6c]

@aaraney any comments on this?

[aaraney]

@aaraney any comments on this?

Looks good! Thanks for tagging me!

[jarq6c]

These tests passed locally and also passed on GH prior to pip version 22.1. The tests on main will pass once #169 is merged in.