Closed nicolasfoss closed 4 days ago
Since people have already been using the data for a couple days, I remove the column from the dictionary. Thanks for the catch, but changing things can break a lot of people's code, so we try to leave things alone as much as possible once a dataset is released!
@jonthegeek Sounds good, I will just get that added on my end and use in my analysis.
@jonthegeek Sorry to be a bother, but I did see you pushed a fix to the readme.md to remove rank
, you might also remove total
as I updated my issue a few times as I caught that one as well. I just want to be helpful and am thankful for your responsiveness. Have a good one!
Hello, Jon:
First, thanks for all the work you put into these. I found a missing column in the
country_results_df
.In your README, you show the following columns for
country_results_df
(see below), but columns 13-14 do not show up in the file we can import for #tidytuesdaycountry_results_df.csv
You will notice that the
total
andrank
columns are missing below.Below is a snippet from the cleaning script in the README, where the
total
andrank
columns are not selected within the select() statement.Here, I will propose a fix, it is simply an update to the select() statement:
After the initial setup, we can manipulate the data and select
total
andrank
along with the rest of the good stuff.We can see below that
total
andrank
are included, and we are off to the...math competition!Thanks again for all your work on the #tidytuesday project, it is such a great opportunity to learn and grow with others.
Created on 2024-09-25 with reprex v2.1.1