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

Add multiple serological codes from rel_dna_ser.txt table's column "Expert Assigned Exceptions". #1070

Open
abragtim opened this issue Nov 8, 2022 · 7 comments
Labels
backend Backend related task/PR blocked Blocked by another PR IKEM

Comments

@abragtim
Copy link
Collaborator

abragtim commented Nov 8, 2022

Add to PARSE_HLA_CODE_EXCEPTIONS_MULTIPLE_SEROLOGICAL_CODES also HLA codes ​​that also have a split_number in "Expert Assigned Exceptions".

viz. discussion in #1048

@abragtim abragtim added the backend Backend related task/PR label Nov 8, 2022
@abragtim abragtim changed the title Add multiple serological codes from rel_dna_ser.txt table's colon "Expert Assigned Exceptions". Add multiple serological codes from rel_dna_ser.txt table's column "Expert Assigned Exceptions". Nov 8, 2022
@abragtim
Copy link
Collaborator Author

abragtim commented Dec 8, 2022

@kubantjan, je to překlep nebo jsme to tak schválně z nějakého důvodu upravili?

Image

@abragtim
Copy link
Collaborator Author

abragtim commented Dec 8, 2022

@kubantjan,
Tak bude vypadat PARSE_HLA_CODE_EXCEPTIONS_MULTIPLE_SEROLOGICAL_CODES po předání expert assigned exceptions. Je to docela velká změna v datech pro multiple values. Máme jistotu v těch expert exceptions od https://hla.alleles.org/wmda/index.html ? To je pro případ, když budeme zadávat expert exceptions nejen pro split assumed. Nebo bude lepší dostávat multiples jenom pro split_assumed a pro ostatní sources tohle nedělat?

C04:03: {'CW6', 'CW4'}
A
02:55: {'A2', 'A28'}
A31:08: {'A31', 'A24'}
A
32:04: {'A32', 'A3'}
A66:02: {'A34', 'A66'}
A
66:05: {'A26', 'A66'}
B15:08: {'B62', 'B75'}
B
15:11: {'B62', 'B75'}
B15:12: {'B62', 'B76'}
B
15:14: {'B62', 'B76'}
B15:15: {'B62', 'B75'}
B
15:19: {'B62', 'B76'}
B15:24: {'B62', 'B77'}
B
15:29: {'B70', 'B15'}
B15:31: {'B62', 'B75'}
B
15:46: {'B62', 'B72'}
B15:52: {'B70', 'B15'}
B
15:80: {'B70', 'B15'}
B15:93: {'B71', 'B15'}
B
15:108: {'B71', 'B15'}
B15:114: {'B70', 'B15'}
B
27:08: {'B7', 'B2708'}
B35:21: {'B78', 'B35'}
B
37:02: {'B37', 'B27'}
B40:05: {'B4005', 'B40'}
B
40:12: {'B48', 'B40'}
B40:21: {'B40', 'B15'}
B
40:26: {'B21', 'B40'}
B40:28: {'B40', 'B4005'}
B
40:39: {'B40', 'B41'}
B44:09: {'B45', 'B44'}
B
44:16: {'B47', 'B44'}
B44:42: {'B21', 'B44'}
B
45:04: {'B21', 'B45'}
B46:11: {'B46', 'B22'}
B
48:02: {'B48', 'B70'}
B50:02: {'B45', 'B50'}
B
53:06: {'B51', 'B53'}
B55:08: {'B56', 'B55'}
B
56:03: {'B15', 'B22'}
B56:06: {'B78', 'B22'}
B
56:10: {'B56', 'B55'}
B56:12: {'B56', 'B55'}
B
58:08: {'B17', 'B5'}
B78:04: {'B78', 'B35'}
C
04:06: {'CW6', 'CW4'}
C12:14: {'CW12', 'CW7'}
C
14:02: {'CW1', 'CW14'}
C14:03: {'CW1', 'CW14'}
C
14:05: {'CW1', 'CW14'}
C14:08: {'CW1', 'CW14'}
C
15:07: {'CW15', 'CW3'}
C15:21: {'CW15', 'CW3'}
C
17:01: {'CW17', 'CW7'}
C17:03: {'CW17', 'CW7'}
C
18:01: {'CW6', 'CW18'}
C18:02: {'CW6', 'CW18'}
DRB1
04:22: {'DR3', 'DR4'}
DRB108:31: {'DR11', 'DR8'}
DRB1
13:21: {'DR11', 'DR13'}
DRB113:24: {'DR11', 'DR13'}
DRB1
13:46: {'DR11', 'DR13'}
DRB113:50: {'DR11', 'DR13'}
DRB1
13:54: {'DR13', 'DR14'}
DRB113:62: {'DR11', 'DR13'}
DRB1
14:15: {'DR8', 'DR14'}
DRB114:53: {'DR13', 'DR14'}
DRB1
14:57: {'DR14', 'DR4'}

Jenom pro split assumed to vypadá takhle:
C04:03: ['CW4', 'CW6']
A
02:55: {'A2', 'A28'}
A31:08: {'A31', 'A24'}
A
32:04: {'A32', 'A3'}
A66:05: {'A26', 'A66'}
B
35:21: {'B78', 'B35'}
B37:02: {'B37', 'B27'}
B
40:21: {'B40', 'B15'}
B40:28: {'B40', 'B4005'}
B
45:04: {'B21', 'B45'}
B46:11: {'B46', 'B22'}
B
53:06: {'B51', 'B53'}
B78:04: {'B78', 'B35'}
DRB1
13:24: {'DR13', 'DR11'}
DRB113:46: {'DR13', 'DR11'}
DRB1
13:62: {'DR13', 'DR11'}
DRB1*14:57: {'DR4', 'DR14'}

@kubantjan
Copy link
Member

@abragtim to 1. je preklep

@kubantjan
Copy link
Member

Tohle je na diskusi s IKEMem to druhe. Pro ted prosim to issue teda pauzneme, ja se ikemu s tim ozvu. Jen prosim v mini PR fixni ten preklep

@kubantjan kubantjan assigned kubantjan and abragtim and unassigned abragtim Dec 16, 2022
@kubantjan
Copy link
Member

PRosim v ramci tohoto PR udelat

  1. smazat ten preklep
  2. pridat test co explicitne testuje tech par kodu co tam mas nahore, jak se ted zparsujou. (tzn mely by se sparsovat tak, ze vratime multiple codes found parsing issue

@kubantjan kubantjan added the IKEM label Dec 16, 2022
@abragtim
Copy link
Collaborator Author

Blocked by #1036

@abragtim abragtim added the blocked Blocked by another PR label Dec 19, 2022
@kubantjan
Copy link
Member

Povazuju spis ted za napad, ktery nekdy muzeme probrat s ikemem, ale spis je to neco co nebudeme delat

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Backend related task/PR blocked Blocked by another PR IKEM
Projects
None yet
2 participants