-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/devsu 2602 add observed variant annotations #440
base: develop
Are you sure you want to change the base?
Feat/devsu 2602 add observed variant annotations #440
Conversation
could you add ticket number to migration file? |
done! |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #440 +/- ##
===========================================
+ Coverage 77.18% 77.24% +0.06%
===========================================
Files 179 181 +2
Lines 6135 6281 +146
Branches 714 732 +18
===========================================
+ Hits 4735 4852 +117
- Misses 1323 1350 +27
- Partials 77 79 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
todo: update includes throughout to get these fields when looking up the variants |
…feat/DEVSU-2602-add-observed-variant-annotations
…tps://github.com/bcgsc/pori_ipr_api into feat/DEVSU-2602-add-observed-variant-annotations
Adds table to store annotations (comment field plus json field similar to kbdata, to store table tags etc) on all variant types. Uses same pivot column as kb-matches table.
Updates rapid-report filtering to check for tags in the annotations field before doing other filtering, so that by updating the tag users can move variants to different rapid-report summary tables.