Skip to content

Added another rule for missing SRI#128

Merged
shilpigoeldev merged 2 commits into
developfrom
feature/36581/Leeds_SRI_missingrule
Nov 12, 2024
Merged

Added another rule for missing SRI#128
shilpigoeldev merged 2 commits into
developfrom
feature/36581/Leeds_SRI_missingrule

Conversation

@shilpigoeldev

Copy link
Copy Markdown
Contributor

What?

We were waiting to hear from lab for gene to be confirmed for one of the missing SRIs from Leeds,(detected during CASREF Refresh) so now it's added in this PR.

Why?

Fiona wanted to be sure if the combination of moleculartestingtype and indicationcategory is meant for MLH1 and asked lab to confirm. Today email has been answered by Rachel from Leeds.

How?

Rules have been updated to include above combination to determine gene.

Testing?

Have run locally and now query return expected result of 3845 -

select count(distinct(servicereportidentifier)) from restricted_results_all where original_filename like '%MMR%';
 count 
-------
  3845
(1 row)

Anything else ?

For reference email dated 22 July 2024 from Fiona "FW: Leeds CRC SRIs" can be seen.

@lauramccluskey1

Copy link
Copy Markdown
Contributor

LGTM

@shilpigoeldev shilpigoeldev merged commit b8245c3 into develop Nov 12, 2024
@shilpigoeldev shilpigoeldev deleted the feature/36581/Leeds_SRI_missingrule branch November 12, 2024 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants