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

CIEL Synonym Search for CHF (Congestive Heart Failure) not working #1283

Closed
jamlung-ri opened this issue Apr 25, 2022 · 6 comments
Closed

CIEL Synonym Search for CHF (Congestive Heart Failure) not working #1283

jamlung-ri opened this issue Apr 25, 2022 · 6 comments
Assignees
Labels
api2 OCL API v2 bug Something isn't working ES Elastic Search openmrs

Comments

@jamlung-ri
Copy link
Contributor

There is a CHF concept that I should be able to find when searching CIEL but currently cannot. Not sure if this is an index issue or something else.

This concept does appear when searching congestive heart failure and when searching another language's synonym, but it never appears when searching CHF.

Loom Video for easy viewing: https://www.loom.com/share/56f5ff9107004b8aad61470c7110529e

@jamlung-ri jamlung-ri added the bug Something isn't working label Apr 25, 2022
snyaggarwal added a commit to OpenConceptLab/oclapi2 that referenced this issue Apr 25, 2022
@snyaggarwal snyaggarwal added api2 OCL API v2 ES Elastic Search labels Apr 25, 2022
@snyaggarwal
Copy link
Contributor

@jamlung-ri This is a bug with search query builder for synonyms. I have addressed this.
It will soon be available on all environments

@snyaggarwal
Copy link
Contributor

@jamlung-ri Fixed on QA/Staging

@jamlung-ri
Copy link
Contributor Author

Seems to be working! The only potential issue I'm noticing is that the concept is only appearing when the casing is correct i.e. "CHF". The concept does not appear when searching "chf". Is that expected? It might not be what CIEL needs.

@snyaggarwal
Copy link
Contributor

Ya we index synonyms in case-sensitive mode (no reason, just the way it was).
In order to do that will have to reindex everything in concepts (around 24 hours in Staging)

@snyaggarwal
Copy link
Contributor

@jamlung-ri this is deployed on all environments and concepts were re-indexed.

@jamlung-ri
Copy link
Contributor Author

This is working well on Production now. I'll go over it with Andy but for now, let's consider this closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api2 OCL API v2 bug Something isn't working ES Elastic Search openmrs
Projects
None yet
Development

No branches or pull requests

2 participants