donaldRwilliams / chkptstanr

Checkpoint Stan R
https://donaldrwilliams.github.io/chkptstanr/
Other
9 stars 5 forks source link

Assumes the existence of Windows binaries #13

Open cpfiffer opened 1 year ago

cpfiffer commented 1 year ago

This line tests for a model named model.exe in the checkpoint folder, and causes errors if the model has already been compiled on a non-Windows system (since this line tests whether model.stan exists).

IMO this whole chunk should be OS-independent, as currently this doesn't work very well on Linux systems. This issue has arisen in other issues (#8 and #6).

It seems like the issue was actually resolved in #6 but for some reason those changes are no longer reflected in the code base?