Open joofio opened 7 months ago
So this is the first draft of the "dumb" automatic preprocessor. It's based on a Tree Search Algorithm to search for certain texts, which are terminologies, but this makes it way faster than its planned AI version. This has been deployed on the dev server for months, so you can use it now. You can see the available preprocessor here
so, if i am not mistaken, this should work?
### preprocessing dumb
POST https://gravitate-health.lst.tfo.upm.es/focusing/focus/bundlepackageleaflet-es-da0fc2395ce219262dfd4f0c9a9f72e1?preprocessors=preprocessing-service-mvp2&lenses=lens-selector-mvp2_pregnancy&patientIdentifier=alicia-1
returning
HTTP/1.1 503 Service Unavailable
content-length: 145
content-type: text/plain
date: Mon, 15 Apr 2024 09:05:00 GMT
server: istio-envoy
x-envoy-upstream-service-time: 7229
connection: close
upstream connect error or disconnect/reset before headers. reset reason: connection failure, transport failure reason: delayed connect error: 111
This is kinda weird, let me check
Sorry I'm looking this now
@joofio Better for testing purposes use
POST https://gravitate-health.lst.tfo.upm.es/focusing/preprocessing/bundlepackageleaflet-es-da0fc2395ce219262dfd4f0c9a9f72e1?preprocessors=preprocessing-service-mvp2
this works for every raw epi right? doesnt work with "Accept: application/json" correct?
its working now as expected with extensions included
POST https://gravitate-health.lst.tfo.upm.es/focusing/preprocessing/bundlepackageleaflet-es-da0fc2395ce219262dfd4f0c9a9f72e1?preprocessors=preprocessing-service-mvp2
Same endpoint
so i have some questions still;
Answers:
PS: preprocessing with bundlepackageleafletxyntha works for me
(sorry i closed this on error)
new list of requirements for this based on discussed today : 4/7/2024
from this ,i can envision the following list of requirements in order of importance:
notes: equal concepts can be stored inside the same codeableconcept. So everything related to pregnancy is attached to a single class name and 1 or more codes.
@aalonsolopez @amedranogil something i might have forgotten? havent tried the current preproc but will do asap and update this if needed.
we can turn this list into Issues in the preprocessor repo, so we can track the progress and discuss each point.
i wanted to test the current preproc before that. give me a day or so
so i tested with the current preprocessor. It lacks in terms that it founds (and some terms dont seem usefull (like Possible?) , and adds a lot for the same concept. example:
{
"url": "elementClass",
"valueString": "Pregnancy"
},
{
"url": "concept",
"valueCodeableReference": {
"concept": {
"coding": [
{
"system": "http://snomed.info/sct",
"code": "11082009",
"display": "Pregnancy"
}
]
}
}
}
],
[
{
"url": "elementClass",
"valueString": "Pregnancy"
},
{
"url": "concept",
"valueCodeableReference": {
"concept": {
"coding": [
{
"system": "http://snomed.info/sct",
"code": "416413003",
"display": "Pregnancy"
}
]
}
}
}
],
This not only creates a ton of different extensions for no reason and the display is not as in the Code System. CodeableConcept in FHIR is 1..* which means it can store several codes for the same concept.
So, taking the example above, it should look like
{
"url": "elementClass",
"valueString": "Pregnancy"
},
{
"url": "concept",
"valueCodeableReference": {
"concept": {
"coding": [
{
"system": "http://snomed.info/sct",
"code": "11082009",
"display": "Pregnancy"
},
{
"system": "http://snomed.info/sct",
"code": "416413003",
"display": "Pregnancy"
}
]
}
}
}
],
and this assuming they are the same concept. For example in the case above, the code 11082009 is abnormal pregancy and not pregnancy (which is quite different ) https://www.findacode.com/snomed/11082009--abnormal-pregnancy.html also, the 416413003 is Advanced maternal age gravida (which is better but still not ok...) https://www.findacode.com/snomed/416413003--advanced-maternal-age-gravida.html
So, it this, for starters, we need to correct the codes and the idea that the same concepts can be stored inside the same codeableconcept
Other minor stuff:
todos from my previous point:
helpfull @aalonsolopez ? let me know