#10

Readme code fails when trying to plot the 'toyota_red' object. Make sure that the object is created with visible code.

part of openjournals/joss-reviews#4740

Add feature to make file_path also backward compatible

As part of (openjournals/joss-reviews#4740)

dont re-iterate the devtools install commands in subsequent readme demo sections. Readme usage sections should assume the tools was installed at the beginning, and not have install bits on top of every code section. It can be confusing for novices to think the tool needs several re-installs.

I'd recommend renaming the _print functions to _latex. This will make it more clear that the functions create latex output.
Also, it creates a clear distinction from R print methods, that other developers and more advanced users might be familiar with, which is quite different.

as part of openjournals/joss-reviews#4740

It appears that the overview_plot function is not plotting correctly. In version 0.0.1.1, the following code does not plot correctly:
overview_plot(dat = toydata, id = ccode, time = year, xaxis = "Time Frame", color = "blue") + labs(title = "Hello World")

The output returns an x-axis label that reads "blue" and the line graphics in the plot are not colored correctly. I did load ggplot2 along with a labs geom to return a title. It is not known if adding this geom has contributed or is the principal cause of the issue.

Venn diagram might not be producing the expected result - it's recommended to use "bar" instead

👋🏽 I maintain the cran checks badges. Please change to the new cran checks badge URL (e.g., https://badges.cranchecks.info/worst/dplyr.svg). Old badges at (e.g. https://cranchecks.info/badges/worst/dplyr) will be unavailable as of Jan 1st 2023.

• Blog post from June 2022 announcing the change: https://recology.info/2022/06/cran-checks-badges/
• New badges repo: https://github.com/r-hub/cchecksbadges
• Old badges repo: https://github.com/sckott/cchecksapi

Add an option to reverse the y-axis in overview_plot() or to allow for alphabetic sorting. Could also be the default.

as art of your JOSS review (openjournals/joss-reviews#4740), I'd like to see some community guidelines in your readme on

1. Where to report bugs
2. How to contribute to the package
3. How to seek support

Write unit tests for new functions

Hi!

As part of your submission to JOSS (openjournals/joss-reviews#4740), I'd recommend adding a section to your paper (and maybe even the readme) comparing your package functionality to packages with similar or overlapping functionality, like skimr, DataExplorer and naniar. I think this will give new users an idea of what this package might offer them beyond the options they might already have.

Congrats on a great package. A feature request if the ime argument could take a list of time unit or ... like in dplyr::select for example.
So something like

overview_plot(dat = toydata, id = ccode, time = year, month)

Update pkdgown website to make it consistent with newest developments

It is requested that two critical arguments be added to this plot function as summarized below:

• Add a color argument that allows the user to select the color to be added to the plot
• Add a size argument that allows the user to modify the size of the geom points

In the current version of the plot, the geom points are too small and should be user-controlled for size.

Readme code fails when trying to plot the 'toyota_red' object. Make sure that the object is created with visible code.

part of openjournals/joss-reviews#4740

This is part of the openjournals/joss-reviews#4740 .

I was wondering why there is a need to specify both destination folder and filename for saving a file? This is quite uncommon when saving files in R, where we would usually provide the entire path as a single argument.

To increase the speed (in larger data sets) implement data.table in the main functions.

For instance see here: https://stackoverflow.com/questions/8713994/venn-diagram-proportional-and-color-shading-with-semi-transparency

Using the forward pipe %<>% is uncommon, and might confuse novice users.
I strongly recommend switching the readme part using this to using the 'normal' pipe.