Skip to content
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

Indicate allele vs antigen mismatch #884

Closed
mmelchers opened this issue Feb 13, 2023 · 7 comments
Closed

Indicate allele vs antigen mismatch #884

mmelchers opened this issue Feb 13, 2023 · 7 comments
Labels
algorithmic Involves a change to the algorithmic logic of Atlas, likely needing approval from HLA experts. enhancement New feature or request matching-algorithm Work relates to the algorithm which matches and scores donors for given patient HLA wmda-mvp [Deprecated] Work required for the algorithm to be acceptable for use by the WMDA as MVP
Milestone

Comments

@mmelchers
Copy link
Collaborator

Title
As a search coordinator I want to know whether a mismatch is an antigen or an allele mismatch. This is especially relevant for cord searches.

Description
In WMDA Search and Match we need to indicate whether a mismatch is an antigen or allele mismatch. ATLAS currently does not provide that info so we always set it to "Antigen", which may confuse some search coordinators. Antigen vs allele mismatch should be based on serology. Please check this page on what should and should not be considered an antigen match: https://share.wmda.info/x/soAZGQ

Component(s) to enhance (if known)
?

Business value (select one or more):

  • Improved user experience

How would this enhancement provide this value?
Search coordinators are better able to determine the best cord search result.

Acceptance Criteria
GIVEN a search in ATLAS has successfully finished
WHEN a certain record in the search results has a mismatch
THEN the type of mismatch (antigen vs allele) is indicated along with its position

Required By Date (if any)
Please explain the significance of the date, if provided.

  • E.g., "Required by <date> ahead of compliance inspection"
@mmelchers mmelchers added the enhancement New feature or request label Feb 13, 2023
@zabeen
Copy link
Contributor

zabeen commented Feb 13, 2023

@mmelchers is there a required by date for this issue?

@zabeen zabeen added matching-algorithm Work relates to the algorithm which matches and scores donors for given patient HLA algorithmic Involves a change to the algorithmic logic of Atlas, likely needing approval from HLA experts. wmda-postmvp Work requested by WMDA which were not part of the original MVP backlog. labels Feb 13, 2023
@mmelchers
Copy link
Collaborator Author

@zabeen I am not sure. I know for some organisations it can be very important. It would definitely help adoption if this feature is present before we go live for everybody.

@mmelchers mmelchers added the wmda-mvp [Deprecated] Work required for the algorithm to be acceptable for use by the WMDA as MVP label Mar 7, 2023
@zabeen zabeen self-assigned this Mar 15, 2023
@zabeen zabeen added this to the v1.6.0 milestone Mar 15, 2023
@zabeen zabeen removed the wmda-postmvp Work requested by WMDA which were not part of the original MVP backlog. label Mar 15, 2023
@zabeen
Copy link
Contributor

zabeen commented Mar 15, 2023

HLD

This involves extending scoring to compare the equivalent serologies of patient-donor typings. The data needed to do this is already being extracted from the IMGT/HLA file, rel_dna_ser. So it should be a fairly isolated change to use that data within scoring to assess whether a P-group mismatched typing is antigen-matched.

I have considered two options for providing the info of whether an allele mismatch is antigen matched/mismatched:

  1. Add a new grade to the MatchGrade enum of AntigenMatchedMismatch, which would be listed between the existing Mismatch grade (which would now mean "allele mismatch AND antigen mismatch" and PermissiveMismatch grade.
  2. Do not extend MatchGrade enum, but instead extend the LocusPositionScoreDetails model with a new bool property: IsAntigenMatch. This would always have the value of true, except when the MatchGrade is Mismatch AND the mismatch is also an antigen mismatch.

I am going with solution 2, as there is a risk of breaking existing logic with solution 1 (either within Atlas or within AN search which is already using Atlas scoring) as it would change the meaning of the grade Mismatch.

@zabeen
Copy link
Contributor

zabeen commented May 3, 2023

Testing Notes

  • Proper manual testing of this feature requires HLA and matching knowledge.
  • Automated BDD tests have been included in the commit which cover the cases listed on the WMDA share page.
  • Therefore, I think for AN testing it is sufficient to simply check whether or not the new property: isAntigenMatch, is now being set within the search results on scored loci.
  • Further UAT can then be conducted on WMDA test environment, as required by @mmelchers.

@zabeen zabeen assigned DmitriyShcherbina and unassigned zabeen May 3, 2023
@DmitriyShcherbina
Copy link

@zabeen Testing status: ok
There is a new isAntigenMatch parameter in Search results of my requests

Image

@zabeen zabeen assigned mmelchers and unassigned DmitriyShcherbina May 3, 2023
@zabeen
Copy link
Contributor

zabeen commented May 22, 2023

Hold off on WMDA testing until analysis of exercise 2 #964 has been completed

@zabeen
Copy link
Contributor

zabeen commented Jul 19, 2023

As exercise 2 involved validation of this work, and change has been in UAT for some time, I am going to assume it is working as expected, and close this ticket. Any bugs regarding this feature can be raised as a new issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
algorithmic Involves a change to the algorithmic logic of Atlas, likely needing approval from HLA experts. enhancement New feature or request matching-algorithm Work relates to the algorithm which matches and scores donors for given patient HLA wmda-mvp [Deprecated] Work required for the algorithm to be acceptable for use by the WMDA as MVP
Projects
None yet
Development

No branches or pull requests

3 participants