Support prefixes in search: eq, ne, gt, lt, ge, le

[jingtang10]

Is your feature request related to a problem? Please describe.
https://www.hl7.org/fhir/search.html#prefix

Describe the solution you'd like
Design and implement the change to the search API

Describe alternatives you've considered
NA

Additional context
NA

Would you like to work on the issue?
No

[epicadk]

#436 & #419 add these prefixes for number and date. Is this what you had in mind or it is something else?

[epicadk]

Most of these are supported for Date and Number ( see #436 and #419 ) I'll add them for quantity types.

[epicadk]

Update, working on this. Will open a pr soon. The challenging thing would be the a canonical match. For eg, a search for 5 grams should match 5000 mg. I think in order for this to work we'd have to make changes to the indexing code. Possibly add another column that stores the value in base units and then at the time or search convert the quantity into base units to search.
I think this library could help us achieve this. I don't think there is another way to convert random UCUM units to base units. However I can't seem to find documentation for the library and I'm not even sure if it is still under active development.

[Tarun-Bhardwaj]

@epicadk, thanks for the update. Is there an ETA you have for completing the code changes for this?

[epicadk]

I'm still searching for a library/any other solution that would help convert units to UCUM base units. I can get the rest of it done in around a week i.e. search on quantity types without canonical matches.

[Tarun-Bhardwaj]

@epicadk , were you able to move ahead with this issue? Any ETA that you can provide for completion?

[epicadk]

Sorry, slipped my mind, I'll open a pr that adds basic functionality by latest tomorrow.

[Tarun-Bhardwaj]

Thanks @epicadk. By adding basic functionality, are there any changes that are being excluded from the PR for now or would it cover the entire scope mentioned as part of this issue? I assigning it to a reviewer in the meantime.

[epicadk]

Yup, all functionality has been added except for date~approximate as discussed here. #419 (comment). Will discuss with @jingtang10 and get that done as well. After that , it should not take more than a day to complete.

[epicadk]

FYI #568

[epicadk]

Re-opened because still need to address #568. We could also close this and just have #568. @Tarun-Bhardwaj

[Tarun-Bhardwaj]

Re-opened because still need to address #568. We could also close this and just have #568. @Tarun-Bhardwaj

@epicadk , thanks Aditya. Let's keep this issue open until #568 is closed as both these issues are closely linked. Have assigned it to you. CC @jingtang10