|
1 | 1 | ## Overview |
2 | 2 |
|
3 | | -This is a resubmission that addresses several action items requested by the CRAN |
4 | | -team after our first submission in April. While we have directly resolved most |
5 | | -of the CRAN requests, we wish to highlight two issues that require some |
6 | | -additional context: |
7 | | - |
8 | | -1. We were asked to provide DOI links to methods references in our Description. |
9 | | -However, we not have any appropriate references, given that this is a conceptually |
10 | | -simple graphics extension package. |
11 | | - |
12 | | -2. In addition to ensuring that all Examples restore the user's original `par` |
13 | | -settings upon completion (which we have addressed), we were asked to ensure that |
14 | | -a user's `par` settings are immediately restored upon exiting our main |
15 | | -`tinyplot` function. However, this more stringent requirement would negate a |
16 | | -main feature of the function---and the **tinyplot** package at large---since, |
17 | | -for example, we need to adjust `par` settings to add legends outside of the plot |
18 | | -margin. Restoring `par` settings by default would mean that users could no |
19 | | -longer add elements like `abline` to a newly created tinyplot in a safe |
20 | | -or consistent way, since the underlying coordinate system has been reverted. |
21 | | -After some back and forth with Prof. Kurt Hornik (see email correspondence with |
22 | | -Achim Zeileis), we understand that this requirement is at least partially |
23 | | -motivated by a lack of precedence. We have thus taken the following steps to |
24 | | -motivate, safeguard, and document the default **tinyplot** behaviour with |
25 | | -respect to `par`: |
26 | | - |
27 | | - - Users can immediately restore their original `par` settings directly by |
28 | | - invoking the `tinyplot(..., restore.par = TRUE)` argument. |
29 | | - |
30 | | - - If users need to revert `par` settings after the fact---i.e. after a |
31 | | - tinyplot has already been drawn---then we provide an additional |
32 | | - `get_saved_par()` convenience function for doing so. This function retrieves |
33 | | - the state of the user's `par` settings from immediately before the preceding |
34 | | - `tinyplot()` call, which are automatically saved in a hidden environment |
35 | | - variable, and can then be used to restore `par` from this original state. |
36 | | - The `?get_saved_par` help documentation provides several examples, as well |
37 | | - as a detailed explanation of our rationale (need) for persisting `par` |
38 | | - settings by default to ensure a consistent user experience. We have |
39 | | - similarly documented steps to revert or control `par` settings in the help |
40 | | - documentation of other **tinyplot** functions. |
41 | | - |
42 | | -Again, we realise that we are in somewhat uncharted waters here without obvious |
43 | | -precedent. But we hope these additional functions and our explicit documentation |
44 | | -will meet the CRAN requirements. We are happy to discuss further with the CRAN |
45 | | -team if that would prove helpful. |
46 | | - |
47 | | -Original submission notes: |
48 | | - |
49 | | -- This is a new package that provides a lightweight extension of the base R |
50 | | -graphics system (including automatic legends for grouped data, facets, etc.) |
51 | | - |
52 | | -- We run a comprehensive test suite as part of our CI development workflow on |
53 | | -GitHub, which includes hundreds of test snapshots (i.e., SVG images). However, |
54 | | -we have removed these tests from our CRAN submission since their volume results |
55 | | -in a large install tarball, beyond CRAN's recommend size limits. |
| 3 | +This is a minor update that introduces support for additional plot types, as |
| 4 | +well as some internal improvements. |
56 | 5 |
|
57 | 6 | ## Test environments |
58 | 7 | Local: Arch Linux |
59 | 8 | GitHub Actions (ubuntu-22.04): release, devel |
60 | 9 |
|
61 | 10 | ## R CMD check results |
62 | 11 |
|
63 | | -0 errors | 0 warnings | 1 note |
64 | | - |
65 | | -* This is a new release. |
| 12 | +0 errors | 0 warnings | 0 notes |
0 commit comments