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

Report Metric Development relies on datTest which is created in CODE/31 #99

Open
stephlocke opened this issue Feb 26, 2017 · 3 comments

Comments

@stephlocke
Copy link

The report only sources the 30 file, but datTest is created in 31. As a result the report should be updated to source both files to ensure it works in a standalone fashion:

source("CODE/30_glmnet_model.R", echo = FALSE)
source("CODE/31_glmnet_model_evaluation.R", echo = FALSE)
@stephlocke
Copy link
Author

Looks like PR #94 did not involve updating the reports ... a number of the others also expect datTest and glm_pred

@geneorama
Copy link
Member

@stephlocke Yes. Those updates for in PR #94 were in response to something else, and I never got a chance to go back and fix the inputs for the report (which was a work in progress anyway). As such the path from the data to the report is currently broken.

@geneorama
Copy link
Member

@stephlocke Granted, this isn't a very satisfying solution, but you should be able to checkout commit 150e399, and that should allow you to compile the document. So, at the command line git checkout 150e399. Nothing's changed yet in the model, so you'd have the same numbers.

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