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

test na to ze se bude removovat positive match #1209

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

kristinagalik
Copy link
Contributor

@kristinagalik kristinagalik commented May 15, 2023

toto je failed test pred zmenami

image

@kristinagalik kristinagalik linked an issue May 15, 2023 that may be closed by this pull request
@kristinagalik kristinagalik changed the title tests: show that set size changes during iteration test na to ze se bude removovat positive match May 15, 2023
@kubantjan kubantjan requested review from jakubmonhart and abragtim and removed request for kubantjan May 22, 2023 07:03
tests/utils/hla_system/test_crossmatch.py Outdated Show resolved Hide resolved
tests/utils/hla_system/test_crossmatch.py Show resolved Hide resolved
Copy link
Collaborator

@abragtim abragtim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ještě pár drobnosti

tests/utils/hla_system/test_crossmatch.py Outdated Show resolved Hide resolved
tests/utils/hla_system/test_crossmatch.py Outdated Show resolved Hide resolved
tests/utils/hla_system/test_crossmatch.py Show resolved Hide resolved
abragtim
abragtim previously approved these changes May 27, 2023
@abragtim abragtim requested a review from kubantjan May 27, 2023 13:44
def test_positive_crossmatch_theoretical_antibody(self):
# add_theoretical_crossmatch_type changes match_type of theoretical antibodies
# to theoretical, thus these two antibodies will be equal
positive_matches = {AntibodyMatch(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jako tohle by se ale nikdy nemelo stat, ze budou protilatky, co maji stejny hla kod ale ruzny typ preci ne?

Obecne chceme mit v testech realna data a ne nejaky situace, ktery v realite nesmi nastat

Copy link
Contributor Author

@kristinagalik kristinagalik May 29, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

toto je priklad co sa stal, tak je asi problem niekde inde... :/ @kubantjan

mam zalozit novu issue ci to riesit vramci tejto

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aha! tak to urcite zkusme vyresit v ramci teto, ale priorita je nizsi nez ty jine veci :)

@kristinagalik
Copy link
Contributor Author

ak by som nestihala toto dotiahnut, situacia sa deje ked funkcia add_theoretical_crossmatch_type vyzera takto:

def add_theoretical_crossmatch_type(positive_matches: Set[AntibodyMatch]):
    for match in positive_matches:
        if match.hla_antibody.type == HLAAntibodyType.THEORETICAL and match.match_type != AntibodyMatchTypes.UNDECIDABLE:
            positive_matches.remove(match)
            positive_matches.add(AntibodyMatch(match.hla_antibody, AntibodyMatchTypes.THEORETICAL))

a ked populatnem s setup-non-empty-db, idem to eventu "Mixed resolution with crossmatch types", dvojica CAN_5, recipientovi pridam antibody DQ[02:03,02:03] s mfi 1000 aj 3000 a potom donorovi pridam antigen DQA1*02:03

@kristinagalik
Copy link
Contributor Author

kristinagalik commented Jun 3, 2023

image
tak sa to deje pri takejto situacii (jeden kod je tam ako aj split aj high res) no neviem - nemame nikde asi osetrene ze ked sa toto napise ze to hodi daky warning alebo co a neviem ci je to proste len moj situacny error a mame sa spoliehat na to ze to tam nikto nebude pisat alebo je lepsie raisovat warning? moreover jedna skupina ma teraz 3 kody (miesto max 2) a nepise to warning

@kubantjan
Copy link
Member

Problemy

  • nemeli bychom povolovat 3 kody dqA proc to neni error
  • kdyz budou jen 2 kody a oba dqa1 ve splitu, tak bude tenhle error taky? obecne prozkoumat teda ten case detailne.

@abragtim abragtim removed their request for review June 19, 2023 08:53
@jakubmonhart jakubmonhart removed their request for review January 29, 2024 15:47
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.

test na to ze se bude removovat positive match
3 participants