Skip to content

Comments

fix!: community models should not inherit from Statement / EvidenceLine#22

Merged
korikuzma merged 9 commits intomainfrom
issue-21
Mar 28, 2025
Merged

fix!: community models should not inherit from Statement / EvidenceLine#22
korikuzma merged 9 commits intomainfrom
issue-21

Conversation

@korikuzma
Copy link
Collaborator

close #21

  • Use model validator instead of inheriting from Statement or EvidenceLine
    • VariantOncogenicityFunctionalImpactEvidenceLine and VariantPathogenicityFunctionalImpactEvidenceLine should NOT inherit from EvidenceLine
    • VariantOncogenicityStudyStatement, VariantPathogenicityStatement, VariantDiagnosticStudyStatement, VariantPrognosticStudyStatement, and VariantTherapeuticResponseStudyStatement should NOT inherit from Statement

…eLine`

close #21

* Use model validator instead of inheriting from `Statement` or `EvidenceLine`
  * `VariantOncogenicityFunctionalImpactEvidenceLine` and `VariantPathogenicityFunctionalImpactEvidenceLine` should NOT inherit from `EvidenceLine`
  * `VariantOncogenicityStudyStatement`, `VariantPathogenicityStatement`, `VariantDiagnosticStudyStatement`, `VariantPrognosticStudyStatement`, and `VariantTherapeuticResponseStudyStatement` should NOT inherit from `Statement`
@korikuzma korikuzma added bug Something isn't working priority:high High priority labels Mar 28, 2025
@korikuzma korikuzma requested a review from larrybabb March 28, 2025 01:37
@korikuzma korikuzma self-assigned this Mar 28, 2025
Copy link
Contributor

@larrybabb larrybabb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work. LGTM

@korikuzma korikuzma requested a review from larrybabb March 28, 2025 13:22
@korikuzma
Copy link
Collaborator Author

@larrybabb My sleep brain got confused with both CCV and Clin_GEN systems starting with ClinGen lol

@korikuzma korikuzma merged commit 841b791 into main Mar 28, 2025
8 checks passed
@korikuzma korikuzma deleted the issue-21 branch March 28, 2025 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working priority:high High priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix the way we're handling constraints

2 participants