Remove factors from 05-data-structures-part2.Rmd

[hannaheow]

We removed the factors section of Data Structures Part 2 consistent with the changes made to Data Structure Part 1 which have already been merged. #737

We recommend adding a section about factors to a lesson about plotting.

This is related to Issue #633.

@sstevens2

[github-actions]

Thank you!

Thank you for your pull request 😃

🤖 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:

• 🎯 correct output
• 🖼️ correct figures
• ❓ new warnings
• ‼️ new errors

Rendered Changes

🔍 Inspect the changes: https://github.com/swcarpentry/r-novice-gapminder/compare/md-outputs..md-outputs-PR-829

The following changes were observed in the rendered markdown documents:

 05-data-structures-part2.md | 290 ++++++++++----------------------------------
 md5sum.txt                  |   2 +-
 2 files changed, 63 insertions(+), 229 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.

⏱️ Updated at 2023-05-12 12:29:28 +0000

[matthieu-bruneaux]

Thanks a lot for your contribution @hannaheow! It is great to have this episode consistent with the previous one in its use of character vectors instead of factors.

I took the liberty of adding a few minor changes to the PR:

• removing mentions of removing NAs since we do not use na.omit() in this episode anymore
• updating the note about removing several rows at once
• updating the numbering of the challenges
• replacing summary(gapminder$country) by summary(gapminder) to be consistent with the text (this was not related to a change you made in the PR)

I also agree with you that it could be interesting to add a section about factors in a lesson about plotting. Would you mind opening a new issue about this?

LGTM for merging (cc @skanwal or @naupaka for confirmation).

[skanwal]

Thanks @hannaheow for your suggestions and inputs. It looks good to me.
Thanks again @matthieu-bruneaux for keeping up with the PRs.