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

MedTagger Output #21

Open
MayeeshaHumaira opened this issue Nov 19, 2022 · 5 comments
Open

MedTagger Output #21

MayeeshaHumaira opened this issue Nov 19, 2022 · 5 comments

Comments

@MayeeshaHumaira
Copy link

Hello,
I am trying to detect Delirium using MedTagger. So I have included the path of the delirium rules. But when I see the output I see MedTagger is detecting "cough" as the delirium-related word. So my question is does the MedTagger use only the rules I gave it or other rules related to different diseases alongside the rules I stated to give an output?

@nadimpas
Copy link

Hello,

Even I am getting the same issue.
It is detecting other diseases even though delirium rules are given. Any idea or help on this ?

@qqndrew
Copy link
Member

qqndrew commented Nov 23, 2022

Hi,
This issue is actually known/we were working on a fix, basically the PASC aho-corasick lookup was hardcoded in by default and we were investigating why.
This should now be fixed as of MedTagger v1.0.49

@nadimpas
Copy link

Hey @qqndrew , The latest version does not process the output. The output file does not contain the predictions.
Can you please check, I am trying to use Delirium rules from AgingNLP/delirium as ruledir.

@qqndrew
Copy link
Member

qqndrew commented Nov 23, 2022

@sunyangfu would be better able to help you with above issue

@sunyangfu
Copy link
Member

Yes, I have scheduled a time with @nadimpas to go over the issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants