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

Spurious error warning re pkgbuild #318

Open
pearsonca opened this issue Nov 8, 2024 · 2 comments
Open

Spurious error warning re pkgbuild #318

pearsonca opened this issue Nov 8, 2024 · 2 comments

Comments

@pearsonca
Copy link

In using odin for the paramix demo analysis, I'm seeing a non-breaking error message:

Loading required package: odin
Loading required namespace: pkgbuild
Failed with error:  ‘there is no package called ‘pkgbuild’’

This is from running the simulate step, which has a separate script to load / build the model.

Unclear how we should be avoiding seeing this spurious (I think?) error, or if there is some library bug which is erroneously raising this for our use case.

@pearsonca pearsonca changed the title Spurious error warning Spurious error warning re pkgbuild Nov 8, 2024
@richfitz
Copy link
Member

richfitz commented Nov 8, 2024

I only see one place we non-fatally load pkgload and that's in can_compile so this may be coming from pkgload - a reprex might make that clearer.

I should warn you - we're about to do an extremely disruptive change that you might want to get ahead of (that may fix that issue). We'll be swapping in the new version of odin soon (probably early Q1 2025) which has minor breaking changes to model code but significant changes to how models are run. See https://mrc-ide.github.io/odin2/articles/migrating.html and https://mrc-ide.github.io/dust2/articles/migrating.html for details.

@pearsonca
Copy link
Author

Yeah, I've been trying to cook up a minimal example, but I also noticed after submitting this the impending odin2 overhaul. Suspect that'll either totally disappear this problem or otherwise reshape it into a different manifestation, so ... balancing my priorities.

To be somewhat cheeky, that package analysis should just work as a reprex (...after much renv gnashing of teeth), but I realize that isn't all that helpful.

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

No branches or pull requests

2 participants