CDCgov / Rt-without-renewal

https://cdcgov.github.io/Rt-without-renewal/
Apache License 2.0
13 stars 2 forks source link

Add plan for all results figures #24

Closed seabbs closed 1 week ago

seabbs commented 5 months ago

As title. As part of this work will need new issues to be created for the desired output figures

seabbs commented 1 month ago

This might be good too address before we start knocking out prototypes

seabbs commented 1 month ago

@SamuelBrand1 I think you are thinking about this now?

SamuelBrand1 commented 3 weeks ago

After some f2f chat with @seabbs, here is my plan for first three figures:

[^1]: @seabbs finds it more common to plot a rolling nowcast as a ribbon plot, without strong favour to that approach. I think this blurs the idea of how new information also changes past inferences. I'd prefer to have a ribbon plot as representing the inference over time made by a model at some inference time $T$, and have a sequence of box plots as representing what the right now estimate of the model at a sequence of $T$ values. The idea here is to keep these similar, but distinct, concepts separate visually.

seabbs commented 3 weeks ago

Nice. I edited 1. to reflect my understanding of what we discussed (that we would plot all horizons (+ and -) for a single estimation date). Do you agree or did you have a different takeaway?

SamuelBrand1 commented 3 weeks ago

No that makes sense, and was what I intended. I've done (bolded) edit for clarity.

seabbs commented 1 week ago

Do you think we have this plan out/moved to their own issues sufficiently to close this?

SamuelBrand1 commented 1 week ago

Yes.