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
Add REVEL Score #242
Comments
As discussed in Teams, We should change the Requirements as follows.
As for the change/design proposal please update this section towards what we discussed, i.e., no full inclusion in varfish db downloader but Snakefiles, more as an add-on tool to annotate extra annotations based on a chr/pos/ref/alt TSV. Annotating extra annotations with the REVEL score would be special case of such a general purpose tool. Please also propose where to change varfish-server with regards to the requirements 1-4. This will most probably be starting points for an design/architecture change document but please provide some structure and start looking at the source code to identify suitable parts in the VarFish Server views/queries/etc. IOW: Please rework your requirements. To answer your question from Teams, we will not have the REVEL score in the mainline of varfish-db-downloader. Rather, we should (1) create a tool to annotate extra annos TSV file with further scores from TSV/tabix files such as downloaded REVEL score and (2) provide users with instrutions on how to extend their extra annos TSV score with REVEL scores for example. We further need a more refined plan of the required changes in varfish-server. I don't see a need for a new table and query engine performance is not so critical, we will need to augment then design. Please start by studying the query engine source code. |
@holtgrewe @snesic Specifications are looking good to me. Just some comments: Tool: I would emphasize, for the tool interface please use argparse (code can be copied from other tools in the DB downloader, e.g. https://github.com/bihealth/varfish-db-downloader/blob/master/tools/knowngeneaa.py ). Also, please locate the tool in the Query: Getting into the query structure can be daunting at first, but the actual change is not too complicated. I wondered if in the long term there could be a generic or automatic solution to add extra annotation fields to the query. Tests: For the planning, setting up tests will probably take at least as long as the actual implementation, if not longer. |
Squashed commit of the following: commit 5a0ff15 Author: Manuel Holtgrewe <manuel.holtgrewe@bihealth.de> Date: Wed Mar 9 11:14:51 2022 +0100 Documenting release cycle and branch names commit 7c50327 Author: Manuel Holtgrewe <manuel.holtgrewe@bihealth.de> Date: Wed Mar 9 11:13:04 2022 +0100 Capping max. number of cases to query at once (varfish-org#372) (varfish-org#377) commit cc29127 Author: Manuel Holtgrewe <manuel.holtgrewe@bihealth.de> Date: Wed Mar 9 08:41:46 2022 +0100 Fixing problem with sodar-core upgrade (varfish-org#375) (varfish-org#376) commit 034c311 Author: Manuel Holtgrewe <manuel.holtgrewe@bihealth.de> Date: Mon Mar 7 15:52:05 2022 +0100 Optimize retrieving query IDs via API (varfish-org#371) (varfish-org#373) commit bdb49c8 Author: Manuel Holtgrewe <manuel.holtgrewe@bihealth.de> Date: Mon Mar 7 15:40:35 2022 +0100 Fix to support sodar-core v0.10.10. commit be2ed28 Author: Manuel Holtgrewe <manuel.holtgrewe@bihealth.de> Date: Mon Mar 7 14:41:25 2022 +0100 Improving performance for fetching result queries (varfish-org#371) (varfish-org#372) Closes: varfish-org#371 commit c0fbd74 Author: Manuel Holtgrewe <manuel.holtgrewe@bihealth.de> Date: Mon Mar 7 11:40:52 2022 +0100 Fixing minor bug in _ClosingWrapper. commit 00ab43f Author: Manuel Holtgrewe <manuel.holtgrewe@bihealth.de> Date: Fri Mar 4 17:24:31 2022 +0100 Documenting Clinical Beacon v1 protocol. commit b954c1e Author: Manuel Holtgrewe <manuel.holtgrewe@bihealth.de> Date: Fri Mar 4 16:48:19 2022 +0100 Changing branch for badge to main commit 005f7d4 Author: Manuel Holtgrewe <manuel.holtgrewe@bihealth.de> Date: Fri Mar 4 16:44:36 2022 +0100 Splitting UNION queries in joint mode (varfish-org#241) (varfish-org#352) commit 5e04b4d Author: Manuel Holtgrewe <manuel.holtgrewe@bihealth.de> Date: Fri Mar 4 16:42:18 2022 +0100 Fixing beaconsite queries with dots in the key id (varfish-org#369) (varfish-org#370) Closes: varfish-org#369 commit dc32e7e Author: Manuel Holtgrewe <manuel.holtgrewe@bihealth.de> Date: Fri Mar 4 16:16:01 2022 +0100 Adding REST API for small variants (varfish-org#332) (varfish-org#341) commit bb8a9c1 Author: Manuel Holtgrewe <manuel.holtgrewe@bihealth.de> Date: Fri Mar 4 14:46:31 2022 +0100 Fixing problem with ACMD classification (varfish-org#359) (varfish-org#368) Closes: varfish-org#359 commit fbfc86c Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Date: Fri Mar 4 14:02:42 2022 +0100 Bump url-parse from 1.5.4 to 1.5.10 in /varfish/vueapp (varfish-org#365) Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.5.4 to 1.5.10. - [Release notes](https://github.com/unshiftio/url-parse/releases) - [Commits](unshiftio/url-parse@1.5.4...1.5.10) --- updated-dependencies: - dependency-name: url-parse dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> commit b06b258 Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Date: Fri Mar 4 14:02:36 2022 +0100 Bump follow-redirects from 1.14.7 to 1.14.8 in /varfish/vueapp (varfish-org#356) Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.7 to 1.14.8. - [Release notes](https://github.com/follow-redirects/follow-redirects/releases) - [Commits](follow-redirects/follow-redirects@v1.14.7...v1.14.8) --- updated-dependencies: - dependency-name: follow-redirects dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> commit b017ad2 Author: Svetozar N <snesic@gmail.com> Date: Fri Mar 4 14:01:57 2022 +0100 Updates to developer_installation.rst (varfish-org#340) commit 00bfc89 Author: Manuel Holtgrewe <manuel.holtgrewe@bihealth.de> Date: Fri Feb 25 10:23:55 2022 +0100 Removing `display_hgmd_public_membership` (varfish-org#363) commit 2898ef0 Author: Oliver Stolpe <oliver.stolpe@bih-charite.de> Date: Wed Feb 23 11:30:11 2022 +0100 Extending developers documentation with topics git and running specific tests (varfish-org#362) commit 21a6e75 Author: Oliver Stolpe <oliver.stolpe@bih-charite.de> Date: Tue Feb 22 17:32:47 2022 +0100 Fixing broken case detail page that has comments on svs. (varfish-org#360)
Closes: varfish-org#242 Related-Issue: varfish-org#242 Projected-Results-Impact: none
Feature Requirements:
Add REVEL score to VarFish so that the variants can be sorted and filtered by this score.
Use cases:
The display should be in the same way as population frequency.
Allow filtration by REVEL score threshold, i.e., we need to add it to the filter form.Design Overview
Different strategies to implement this are discussed here. We decided to follow the option where the REVEL score is added to the extra annotations table. We further want to allow annotation of arbitrary values from the extra annotations in the result table (and downloads) and allow filtration by these values.
Steps
At this point, the extra annotations such as the REVEL score is visible to users and users can already sort in the result. We also need to ensure that the export works here.
Care needs to be taken that the results table does not explode in terms of number of columns. Users should be able to show/hide the extra annotation columns similar to how it's done for columns such as "distance to splice site."
Extra Annotation Tresholds Extend the filter form (and supporting) code such that it is aware of the extra annotations and thresholds can be set.Depencies
(3.) ->(2.)Notes
Design Details
1. Extra Annotations Data Update
INPUT
USAGE
RESULT
Affected Components
varfish-db-downloader
with documentation.varfish-db-downloader
workflows or existing files.2. Extra Annotations Into Results
(a) augmenting the query infrastructure
variants.query
such that the information from theExtraAnno
model is available in the resultsExtendQueryPartsCommentsJoin
as a model (copy-paste-and-adjust) =>ExtendQueryPartsExtraAnnoJoin
(b) adjust result table templates
variants/templates/variants/filter_result/{header,row}.html
detail-frequencies-thousand-genomes
)table.html
, look for stringresult-columns-selector
variants.views
(c) for TSV/Excel downloads
variants.file_exports
CaseExporterBase._yield_columns
andCaseExporterBase._yield_small_vars
to return the extra annotations in an appropriate way (break apart the JSON list that will be in the output)(d) adjust documentation
Affected Components
variants.queries
,variants.file_exports
,variants.views
, possibly more modules invariants
variants/templates
3. Extra Annotation ThresholdsThe text was updated successfully, but these errors were encountered: