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

Improve tokenization of icd11 codes in redis index #1838

Closed
sainak opened this issue Jan 19, 2024 · 6 comments · Fixed by #2193
Closed

Improve tokenization of icd11 codes in redis index #1838

sainak opened this issue Jan 19, 2024 · 6 comments · Fixed by #2193
Assignees
Labels
good first issue Good for newcomers

Comments

@sainak
Copy link
Member

sainak commented Jan 19, 2024

icd11 diagnosis codes like RA01.1 are considered as 2 tokens if the string is indexed directly (ref)
so we need to escape the periods like this RA01\.1 before indexing

@Clemo97
Copy link

Clemo97 commented Mar 3, 2024

Hi @sainak I'd like to take on this issue

@Clemo97
Copy link

Clemo97 commented Mar 6, 2024

@sainak I just noticed that his issue might conflict with another issue by @DraKen0009 #1927

@ab1123
Copy link

ab1123 commented Mar 30, 2024

If this issue is already not resolved, I would like to work on this issue!!
@sainak

@Clemo97
Copy link

Clemo97 commented Apr 3, 2024

If this issue is already not resolved, I would like to work on this issue!! @sainak

maybe we can work on it together

@rithviknishad
Copy link
Member

@Clemo97 unassigning due to no activity. Feel free to take it up if you'd be making a PR to solve this issue.

@aparnacoronasafe
Copy link
Member

@sainak Can you please take this up as a priority?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants