-
Notifications
You must be signed in to change notification settings - Fork 43
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
allow filtering for years when creating historical.mif #453
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please, solve the conflicts
@@ -15,7 +13,7 @@ | |||
#' fullREMIND() | |||
#' } | |||
#' | |||
fullREMIND <- function(rev = 0) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
did this rev argument has any functionality in retrieveData, which we are only not using?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing I can see. rev
is an input parameter to retrieveData and is therefore set in our inputdata generation, but it does have to be passed to the full functions, unless you want to use it there. As we do not use it here, I deleted it.
Running input data generation without it went through as expected.
#' @seealso | ||
#' \code{\link{fullREMIND}},\code{\link{readSource}},\code{\link{getCalculations}},\code{\link{calcOutput}} | ||
#' @examples | ||
#' \dontrun{ | ||
#' fullVALIDATIONREMIND() | ||
#' } | ||
#' | ||
fullVALIDATIONREMIND <- function(rev = 0) { | ||
fullVALIDATIONREMIND <- function(years = NULL) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here regradin rev
This PR makes it possible to restrict the years when generating the historical.mif, so we don't need to get all the years starting from the 1700s