Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change likes_string to something less confusing #901

Open
kaijagahm opened this issue Sep 16, 2024 · 1 comment
Open

Change likes_string to something less confusing #901

kaijagahm opened this issue Sep 16, 2024 · 1 comment

Comments

@kaijagahm
Copy link

How could the content be improved?

Several times while teaching this lesson, I've had learners stumble over the column in the cats data frame called likes_string. Especially for learners coming from other programming languages, where there is a data type called "string", this variable name throws them off. They don't immediately think of literal string in relation to cats--instead, they assume that this is a variable called "likes" that has the type "string." Then they get confused by the zeroes and ones, and we have to spend a while explaining that the intention of this part of the lesson is quite different. In fact, when I was taking the lesson for the first time, I also got confused at this part.

I think this is an example of extraneous cognitive load that could be very easily fixed if this variable name were changed to something else with a more obvious meaning, such as likes_catnip or likes_toys or likes_playing. Currently, the variable name is activating learners' prior knowledge in a way that is incorrect and unhelpful.

I'd be happy to submit a PR to fix this, if people like the suggestion! Should be an easy substitution that doesn't change the narrative of the lesson at all.

Which part of the content does your suggestion apply to?

https://swcarpentry.github.io/r-novice-gapminder/04-data-structures-part1.html

@naupaka
Copy link
Member

naupaka commented Sep 20, 2024

I like this suggestion. I think likes_catnip would be great since that's hard to mistake for anything else. I'd be happy to review a PR with this change implemented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants