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

Fix to Jags detection error #88

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

DylanCarbone
Copy link
Collaborator

This fixes an error I introduced with the detection of JAGS. The previous approach relied on runjags::findjags(). This in turn relies on the activation of rjags through library(rjags). However, the library load fails if JAGS is not installed. During CI, rjags could not be loaded as github actions does not have access the jags, and this would cause CI to fail. In summary the JAGS detection could only take place if JAGS was already installed and rjags loaded, but this approach would cause failures in continuous integration as well users without jags trying to load sparta. The new approach simply searches common install locations for the JAGS executable.

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

Successfully merging this pull request may close these issues.

1 participant