snaps are too large for CRAN submission

[adokter]

Hi @CeciliaNilsson709 and @peterdesmet,

I noticed the snaps you have added for plot testing are very large, 85 Mb in total currently. This makes the package size increase from ~ 5Mb previously to almost 10Mb, and this is too large for CRAN (5Mb max).

Is there a way of not including snaps in CRAN submissions?

[peterdesmet]

Can we include these in .Rbuildignore or won't we be able to use them for the tests then?

[adokter]

Added _snaps to .Rbuildignore in commit ee04294a46bc9e88e9cf06fa44d0847f53fcf163. This reduces the package size back to ~5Mb, but unfortunately the tests are now skipped, both on Github and when running local devtools::check(). Hitting the "Run Tests" button in Rstudio for test-plot.vpts.R does still work

[peterdesmet]

Ok, good to know, ignoring might not be the best option then. It’s mainly the vpts snaps that are huge, could reduce x-axis for those.

We can also have a skip_if function for these visual tests, cf. docker, but that doesn’t solve the size issue.

We also still have the option of moving some files (not the snaps) to zenodo.

[adokter]

Besides the snaps we still have a package size issue, we are at 5.7 Mb without snaps, with 5.0 Mb being the max. Reopening #376

[CeciliaNilsson709]

Reducing the x axis on the plots does help a lot, but we are still way too far from being able to include them, especially if we are already over. We could use skip_on_cran() for the visual tests, would that be an option? Not how that would affect code coverage etc..

The example vpts file covers 10 days, we could shorten it? I would think that a shorter time frame would work equally well for most examples?