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

Unclear Purpose for test-zzzz-rxUse.R #29

Closed
billdenney opened this issue Jan 22, 2022 · 2 comments
Closed

Unclear Purpose for test-zzzz-rxUse.R #29

billdenney opened this issue Jan 22, 2022 · 2 comments

Comments

@billdenney
Copy link
Contributor

It looks like test-zzzz-rxUse.R builds a package around a model. Is that a necessary test? It fails on Windows 10.

(The underlying question is that I'm uncertain of why rxUse() exists. And the next question is if/how we should test it.)

@mattfidler
Copy link
Member

rxUse allows rxode2 models to be packaged as libraries of pre-compiled models, like this:

https://github.com/nlmixrdevelopment/rxModels

So if you compile a roxde2 model (not ui at this point) then you can save it and the build process will compile and link against rxode2

So... building a package does make sense in this case, It think.

@billdenney
Copy link
Contributor Author

Ah, okay. That makes sense. The test succeeds on Linux but fails on Windows. I'll look into it a bit more, but I may end up skipping it on Windows.

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

2 participants