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

Add reminder about installing dependencies. #3

Closed
wants to merge 1 commit into from
Closed

Add reminder about installing dependencies. #3

wants to merge 1 commit into from

Conversation

hlapp
Copy link

@hlapp hlapp commented Feb 18, 2014

This may seem trivial to an experienced R programmer, but the default is in fact not to install dependencies. If one forgets to enable that, errors will result later that can be obscure to trace
back to the failure of installing dependencies earlier.

This may seem trivial to an experienced R programmer, but the default is in fact not to install dependencies. If one forgets to enable that, errors will result later that can be obscure to trace
back to the failure of installing dependencies earlier.
@cboettig
Copy link
Owner

And yet again you've encountered an error that is more tricky than it may appear!

Since writing this paper, odesolve has been deprecated on CRAN in favor of deSolve, whose lsoda function does not in fact use the same function call! Because of the branch becoming repo problem I discuss in #2, this change was fixed on all branches of the earlywarning package but never ported here. (which is why this package should have been a new package depending directly on earlywarning.) It is of course an annoying deeper issue that package dependencies can change on CRAN in this way, either by a package vanishing, or worse, just by it's API changing in some update (CRAN doesn't enforce backwards compatible function definitions, and many scientist R developers don't even give that a thought, since many imagine only interactive use of their functions rather than scripts and dependencies).

So I should merge the desolve changes from the earlywarning branch, and update the README and DESCRIPTION accordingly...

@hlapp If you haven't seen it, your experiences in this exercise might be very relevant to the Mozilla Science Lab / Software Carpentry discussion on code reuse, see mozillascience/code-research-object#2 There's a desire to go beyond Publish your code, it's good enough but also recognize the lack of incentive to do this 3x extra work of making research code easily deploy-able and reusable.

@cboettig
Copy link
Owner

Done. Most of this should now be handled well by devtools. Added note to readme about installing populationdynamics too.

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.

2 participants