Closed kaitejohnson closed 1 month ago
I think this is getting really close to being able to remove cfaforecastrenewalww dependency from wweval/the container build! @gvegayon Do you want me to do that in this PR or do you want to work on in a separate branch merged off of lesscfaforecastrenewalww-2?
This PR is not big, so I think we still have some space to do stuff, so I would say, "go for it, @kaitejohnson!"
Ok will test out trying to remove and see how it goes!
This PR closes #176. It does the following:
wweval
to use the one inwwinference::get_draws()
for binding the data to the posterior estimateswweval
packagecfaforecastrenewalww
inwweval
, the container builds, and the pipeline codeTo make sure this works as expected I:
Next steps:
wwinference
targets_eval_postprocessing.R
pipeline, after #179 is also merged. I anticipate there may be a few additional modifications here since a part of this relies on the input data, some of which is now formatted slightly differently (column names differ) . I will change the functions inwweval
accordingly. The scores, quantiles, etc should be formatted the way thatwweval
expects them downstream.