You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Description:
If you run FrEDI with outputlist =TRUE to return a list object with scenario input data, it returns an empty scenario list object. This is on the "main" branch.
Reprex: Set your working directory to main branch FrEDI and run -
library(devtools)
load_all(".")
test <- run_fredi(outputList = TRUE)
test$scenarios
The issue:
On this commit you can see on Lines 406-416, after the default scenario is added to the inputsList if scenario data is missing, the reassignment of inputs to the returnlist object was deleted during the refactoring. In the current main "run_fredi" you'll see on line 689 when we finalize the returnList object, it assigns returnList[["scenarios"]] which only gets initialized on line 219 as an empty list and never gets redefined.
Proposed Solution:
After the check for inputs, and if missing the default scenarios are assigned (L500-L515)-
Check outputslist for TRUE
assign the inputs list to the object lists of statusList, argsList, and returnList across names in the inputsList
The text was updated successfully, but these errors were encountered:
Description:
If you run FrEDI with
outputlist =TRUE
to return a list object with scenario input data, it returns an empty scenario list object. This is on the "main" branch.Reprex: Set your working directory to main branch FrEDI and run -
The issue:
On this commit you can see on Lines 406-416, after the default scenario is added to the inputsList if scenario data is missing, the reassignment of inputs to the returnlist object was deleted during the refactoring. In the current main "run_fredi" you'll see on line 689 when we finalize the returnList object, it assigns
returnList[["scenarios"]]
which only gets initialized on line 219 as an empty list and never gets redefined.Proposed Solution:
After the check for inputs, and if missing the default scenarios are assigned (L500-L515)-
The text was updated successfully, but these errors were encountered: