CDCgov / cfa-viral-lineage-model

Apache License 2.0
10 stars 0 forks source link

Can we shrink forecast plot image sizes? #40

Closed thanasibakis closed 1 month ago

thanasibakis commented 2 months ago

https://github.com/CDCgov/cfa-viral-lineage-model/blob/cbc7aa564641e4a173654f2ffe8f345eab9cfd24/present-day-forecasting/main.py#L105-L111

This image is pretty large in size (dimension 24000 × 18000, taking ~5 seconds to open on a M1 Mac).

Can it be made smaller without impacting visibility?

(Potential hint: I think I should have set the units argument here, and needing to adjust limitsize should have clued me in on this)

swo commented 1 month ago

oh, like this is 30x40 inches?

thanasibakis commented 1 month ago

Indeed 😅

Also, I’m now noticing that my dpi=300 is somehow ignored, and a DPI of 600 is being used, which certainly does not help