search

nlmixrdevelopment / RxODE

RxODE is an R package that facilitates easy simulations in R
https://nlmixrdevelopment.github.io/RxODE/
GNU General Public License v3.0
54 stars 14 forks source link

RxODE version 0.8.1-0 from github seems have a little problem working with nlmixr #78

Closed DedeLLL closed 5 years ago

DedeLLL commented 5 years ago

Hi, Thanks for the great effort on the useful packages-- nlmixr and RxODE. I installed the latest version of RxODE (0.8.1-0) from Github both on Win10 and macos mojave. However, every time I tried to run the model, both systems returned the error: Error in inDL(x, as.logical(local), as.logical(now), ...) : unable to load shared object 'C:/Users/......./......./saem24944ef59a6x64.dll': I removed the package and reinstalled the second latest version 0.8.0-9 from Cran and everything seemed to be back to normal again. Did anyone encounter the same issue?

mattfidler commented 5 years ago

HI @DedeLLL

If you use the RxODE version on github you need to use the nlmixr version on github. The interface between SAEM and FOCEi and RxODE have changed to support the new events:

https://nlmixrdevelopment.github.io/RxODE/articles/RxODE-events.html

mattfidler commented 5 years ago

There are other (breaking) changes in nlmixr too including different objective function values for nlme (to more closely match NONMEM's FOCEi objective function value) and log likelihood adjustment for FOCEi, to give the correct likelihood [by correcting based on the number of observations times some factor (log(2*pi) if I remember correctly)].

DedeLLL commented 5 years ago

HI @DedeLLL

If you use the RxODE version on github you need to use the nlmixr version on github. The interface between SAEM and FOCEi and RxODE have changed to support the new events:

https://nlmixrdevelopment.github.io/RxODE/articles/RxODE-events.html

Thanks a lot, Dr filder, It worked. The RxODE version 0.8.1.0 needs to cooperate with nlmixr version 1.0.0.8 from github. By the way, a little reminding for those interested, installing nlmixr version 1.0.0.8 needs to install RxODE version 0.8.1.0 first or you may got an error in the compilation process during installing.

DedeLLL commented 5 years ago

There are other (breaking) changes in nlmixr too including different objective function values for nlme (to more closely match NONMEM's FOCEi objective function value) and log likelihood adjustment for FOCEi, to give the correct likelihood [by correcting based on the number of observations times some factor (log(2*pi) if I remember correctly)].

Definitely thanks for the great work, such a powerful package deserving to be explored. 👍

mattfidler commented 5 years ago

Thank you @DedeLLL . I'm glad you got it working!