search

datacarpentry / r-raster-vector-geospatial

Introduction to Geospatial Raster and Vector Data with R
https://datacarpentry.org/r-raster-vector-geospatial
Other
113 stars 110 forks source link

Rm group by func call #439

Closed sstevens2 closed 5 months ago

sstevens2 commented 5 months ago

The count() function is an alias for groupby +summarize(n()). It will group by the variable and count it in one function call so including the groupby function is redundant.

github-actions[bot] commented 5 months ago

Thank you!

Thank you for your pull request :smiley:

:robot: This automated message can help you check the rendered files in your submission for clarity. If you have any questions, please feel free to open an issue in {sandpaper}.

If you have files that automatically render output (e.g. R Markdown), then you should check for the following:

Rendered Changes

:mag: Inspect the changes: https://github.com/datacarpentry/r-raster-vector-geospatial/compare/md-outputs..md-outputs-PR-439

The following changes were observed in the rendered markdown documents:

 02-raster-plot.md | 38 +++++++++-----------------------------
 md5sum.txt        |  2 +-
 2 files changed, 10 insertions(+), 30 deletions(-)
What does this mean? If you have source files that require output and figures to be generated (e.g. R Markdown), then it is important to make sure the generated figures and output are reproducible. This output provides a way for you to inspect the output in a diff-friendly manner so that it's easy to see the changes that occur due to new software versions or randomisation.

:stopwatch: Updated at 2024-06-15 20:56:40 +0000

drakeasberry commented 5 months ago

Thanks @sstevens2. Really like the simplicity given by removing the redundant function.