-
Notifications
You must be signed in to change notification settings - Fork 14
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
Interpreting CLSI values (blood) #5
Comments
In addition looking at the https://github.com/msberends/AMR/blob/master/data-raw/DRGLST1.xlsx table I think the values here are for animals |
Looking at the code I believe an additional filter here is required
|
Nice catch! I removed all animal records from the file, and now: data.frame(mo = "E. coli",
AMC = 4) %>%
as.rsi(guideline = "CLSI 2019")
#> NOTE: Using column `mo` as input for `col_mo`.
#> NOTE: Assuming column `AMC` (amoxicillin/clavulanic acid) contains MIC values.
#> => Interpreting MIC values of `AMC` (amoxicillin/clavulanic acid) using guideline CLSI 2019 ... OK.
#> mo AMC
#> 1 E. coli S I'm running checks now and will then upload it to the master branch. I'll let you know when it's done! You're just in time, I was about to release a new version to CRAN! So if you can confirm this is fixed, I will 😉 |
Great - thank you. |
No, just do: remotes::install_github("msberends/AMR") It will install the latest beta version to your computer and then you can just run your code again (you might need to refresh your R session). But this issue is not yet fixed, I'm now also fixing your other issue in one go 🙂 |
📋 Please take part in our survey!Now you're at it, please fill in our short questionnaire! You can fill it in completely anonymously. If you know others using this Your valuable input will help to improve the package and its functionalities 🙂 |
Survey completed |
Let me know when it's ready for testing |
Ready! You can test the 'new' Thanks for filling in the survey! |
Sorry for the late reply. Was checking out my comparison and it's much improved. The fixes for human only and the I was running into some more discrepancies. It turns out that exports from Vitek have values such as <=.25 (i.e with no leading 0). I can solve this by transforming my dataframe as follows
would you consider making a modification to the |
No problem about the late reply, wasn’t that late! Using across() already, nice 😜 Sure thing, I’ll fix it in the morning. New issue would be best, I’ll handle it. Many thanks again! |
Fix released to official R repositoryJust a small notice: we released a new software version, containing the fix to this issue. Please update your version using either the user interface of your software, or by running: install.packages("AMR") |
I noticed a peculiarity with interpretation of some CLSI antibiotics
An MIC value of 4 for AMC which I think should be interpreted as sensitive is being interpreted as R for CLSI 2019
I think this is because of row 6 in the table below
This should not be used for bacteraemia, only skin and soft tissue. Is there any way to exclude these and only include interpretations for blood?
The text was updated successfully, but these errors were encountered: