Feat/algorithm configuration schema

[cbrinson-rise8]

Description

In models.py add new classes for storing data related to the available algorithms to run.

erDiagram
    Algorithm {
        int id
        bool is_default "a check should be added to guarentee that only 1 row in the table is marked as the default"
        string label "should be unique"
        string description
    }

    AlgorithmPass {
        int id
        int algorithm_id
        int[] blockingkeys "a list of values from the BlockingKey table"
        string[] evaluators "a list of matching functions and values to use"
        string rule "the evaluation rule function"
        float cluster_ratio
        json kwargs "extra parameters to pass to the evalator functions"
    }

    Algorithm ||--o{ AlgorithmPass: "has"

Loading

Related Issues

closes #13

Additional Notes

[Add any additional context or notes that reviewers should know about.]

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.
• I have notified teammates in the review thread to build awareness.

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.

[alhayward]

Nice job!

[cbrinson-rise8]

if target.is_default:
        existing = session.query(Algorithm).filter(Algorithm.is_default == True).first()    # noqa

Link to the line of code

Just a note for anyone wondering what # noqa is. It suppreses the ruff linting error on this line. The reason for this is ruff wants to enforce the use of the is instead of the == operator. But using is doesn't seem to work when comparing SQL boolean values.

[ericbuckley]

Looks great, thanks for all your work on this @cbrinson-rise8

[ericbuckley]

@cbrinson-rise8 one more small thing. Can you please add your name to the authors list!