Closed n8thangreen closed 1 year ago
Thank you, @n8thangreen. Just to make sure, in summary.bcea.R
, the line
rownames(EU_tab) <- he$interventions[as.numeric(interv_idx)]
is supposed to fix the issue we discussed the other day (which I had simply fixed by setting the rownames
to he$interventions
?
Thanks!
The slight change I made was to account for when the interventions are not in numeric order.
Also TODO, the original fix for summary.bcea
doesn't work properly when there are a subset of all of the interventions. It still prints the kstar for all of them. I had a look over the weekend and its because kstar
and best
aren't changed in the bcea
call. The thing to do is change new_bcea
but making sure this then doesnt break something else!
Thanks. I'll merge this now!
I just noticed that the dev branch in your repo is over 100 commits behind mine but the main is up to date which is weird! I suppose it would have made more sense to originally synced with your dev and then let you sync with your main rather than to your main directly.