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.

Sign up for GitHub

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

Jump to bottom

ICD-10 API: implement search algorithm #2

Closed
4 tasks done
andreakreichgauer opened this issue Aug 17, 2020 · 1 comment · Fixed by #5
Closed
4 tasks done

ICD-10 API: implement search algorithm #2

andreakreichgauer opened this issue Aug 17, 2020 · 1 comment · Fixed by #5
Assignees
@andreakreichgauer
Labels
Priority: Medium Medium priority issue. Status: Done Something that is finished. Type: Story An improvement to the project from a user's perspective.

Comments

@andreakreichgauer
Copy link
Contributor

andreakreichgauer commented Aug 17, 2020 •
edited

🚀 Describe the Improvement

When entering a new diagnose, I want to simultaneously get a list with matching suggestions.

✔ Goals

To complete this story, these things have to be done:

  • fuzzy search instead of strict string comparison
  • search for matches not only in the icd title but also in meta data (such as icd field "inclusion", containing related diseases and synonyms)
  • allow searching for icd10 codes instead of text only
  • allow searching for multiple terms - regardless of their order

🧰 Possible Solution

  • search will be based on fuse.js
  • searching for multiple terms must include searching for all possible combinations of terms

🚧 Blocked by

Work that needs to be done before this story can be started:

  • #NUM - creation of fields in X
@andreakreichgauer andreakreichgauer mentioned this issue Aug 17, 2020
Closed
4 tasks
@andreakreichgauer andreakreichgauer added Priority: Medium Medium priority issue. Status: Review Needed Please review the corresponding code. Type: Story An improvement to the project from a user's perspective. labels Aug 18, 2020
@andreakreichgauer andreakreichgauer linked a pull request Aug 18, 2020 that will close this issue
Epic/#1 icd10 api #5
Merged
@andreakreichgauer andreakreichgauer self-assigned this Aug 18, 2020
@friedow friedow mentioned this issue Aug 19, 2020
Merged
@friedow friedow added Status: Revision Needed Something that needs to be overhauled. and removed Status: Review Needed Please review the corresponding code. labels Aug 20, 2020
@andreakreichgauer andreakreichgauer added Status: Done Something that is finished. and removed Status: Revision Needed Something that needs to be overhauled. labels Aug 28, 2020
@friedow
Copy link
Contributor

friedow commented Aug 28, 2020

Closing because corresponding PR #5 is merged.

@friedow friedow closed this as completed Aug 28, 2020
@andreakreichgauer andreakreichgauer mentioned this issue Aug 28, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andreakreichgauer andreakreichgauer

Labels
Priority: Medium Medium priority issue. Status: Done Something that is finished. Type: Story An improvement to the project from a user's perspective.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Epic/#1 icd10 api dot-base/icd-10-api
2 participants
@friedow @andreakreichgauer