Feature/36558/st george old import mising records #123
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…now the FS gene in question
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What?
Adding in functionality to use the test status 10 value when recording the status of variant results.
Why?
There are 64 records that are not currently being imported from the raw files into MBIS / CASREF. These records are large;ly abnormal results that which we really do want to capture.
Need to broaden the regex that assigns a teststatus as 2. To include fields that contain strings such as 'IVS' or 'Del X 5-7'.
How?
Added in method to ctreate BRCA1/2 records as standard when we don't know what the FS gene in question is .
Added in method to handle a variant when we don't know the gene it belongs to
Expanded to exon regex to account for additional variant nomenclatures eg. IVS, DEL X 5-7
Testing?