-
Notifications
You must be signed in to change notification settings - Fork 2
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
Problem with DUS step (DrugExposureDiagnosis) #68
Comments
@JavierGorri thank you for opening this issue. However, I do not see the error message and where the error occured from this screenshot (it is not displaying fully). @D229463 Are you sure the renv worked? DrugExposureDiagnostics is not part of the DUS renv. |
@tiozab I try to explain it in more detail. We made three attempts:
> source(here("drugs.R"))
I add some logs: File log that is found in storage folder: Trace of the attempts 1 and 2: Attempt 3 image (it´s the image of @JavierGorri comment) Thanks and regards |
Thanks @D229463 can you show me screenshot where the packages tab shows the "codelistgenerator" package please? |
great, because the checkDbType function comes from Codelistgenerator, can you show me the libraries you load in your codetorun script? @D229463 |
@D229463 the error does not make sense because checkDbType is a function from CodelistGenerator and no one has had this error before. https://darwin-eu.github.io/CodelistGenerator/articles/a04_Vocab_based_codelists.html can you try whether you can reproduce the vignette with your data? |
@tiozab ok, yestyerday we had run the script from the begginig with the space added (now it´s still running) and when it finishes I try to reproduce the vignette |
@D229463 ah I thought your code stopped at the drug cohort generation. That means this error was resolved? |
No, we have tried it again (from the beginning avoiding re-executions)... but as I see in the log it is still in step "GENERATE INDICATION CONCEPTS" (last time it needed 13 hours in this step) |
@edward-burn can you have a look at this please? |
@D229463 can I just confirm, if you clear your environment and restart r, does the following work cdm <- mockVocabRef() ? |
Yea, it work well
|
@D229463 Not really, but with everything anew, please try the DUS step once more with the 1.4 file and then we see where we get :-) we do not assess acetaminophen so maybe there is just s.th. wrong with that ingredient |
We have run the script several times and it fails to create temporary tables on step "GENERATE DRUG CONCEPTS" I add file logs:
It has always failed in the "Created a temporary table" number 52. Log says "subindex out of bounds" but I don't know what index it is referring to |
@D229463 drug 52 is certolizumab. We have had problems with this drug in other places. I have modified the drugs.R script for certolizumab and cytarabine (the two drugs causing problems), can you replace this R script in your repo with this version and try again from "GENERATE DRUG CONCEPTS" |
we have executed the new drugs.R file but we get the same error in the same "create table" :-( we have check that the error is in first step of drugs.R |
@edward-burn may it be a memory issue? |
@tiozab What can we do now? |
Hi, are there some news about the error? |
@D229463 sorry I was on holiday. |
@Theresa. I open a new issue, just to be clear. We have solve the old one (Problem with DUS step)
We have run part of the code (we have done "Indications concepts" and "Indication cohort") but we have a problem with "Drug concepts". This is the problem with "DrugExposureDiagnosis".
@JavierGorri
The text was updated successfully, but these errors were encountered: