-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add an example to incorporate remote clinical pipeline and stave #43
base: master
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #43 +/- ##
==========================================
- Coverage 84.17% 84.14% -0.04%
==========================================
Files 9 8 -1
Lines 512 511 -1
==========================================
- Hits 431 430 -1
Misses 81 81 Help us with your feedback. Take ten seconds to tell us how you rate us. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few comments, but overall, if this example is runnable it is fine with me. We can improve in the future.
Could we leave these two issues metioned in the conversations in the future? I am not familiar with the ontology generation yet so it may take much time. |
Actually, it might be important to understand the following items since they are the key for understanding our visualization:
|
Got it. Actually, I understand how the ontologies related to stave. I didn't understand what you meant at first, I thought it was about writing python code to automatically generate ontologies. So I think what you want me to do is to omit most of the ontologies that were already written, write only the extra ones we added here, and then use the generate ontology function to merge them? |
Yeah I think you got the idea, basically we don’t need to repeat the ontology spec here again, and be able to comtrol the ontology should help you demo |
This PR fixes #42.
Description of changes
This PR is totally based on what has been done by Piyush, here is the reference: https://github.com/Piyush13y/forte/tree/745_clinical_pipeline_stave
The changes include:
clinical_processing_pipeline.py
, and add an argument to supportPlainTextReader()
. In case users may not have the mimic iii datasets, they can just use the sample data in thesample_data/notes.txt
for the function test.README.md
, according to the errors or obstacles I encountered when I set up the connection between the pipeline and stave.Bugs encountered
Initially, the version of Elasticsearch I downloaded is 8.2.2, and I failed the setup of this according to the tutorials. ( Many unsolvable bugs). When I change the version to 7.12.2 (given in
README.md
), everything works perfectly fine.