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

fix: pseudocode lesson updates - p1 #78

Merged
merged 4 commits into from
Oct 24, 2024
Merged

Conversation

lwasser
Copy link
Member

@lwasser lwasser commented Oct 24, 2024

Ok - i rewrote the pseudocode lesson!! this is part 1 of it.

In the first section, I had them think through processing some code and the basic steps, but there are no functions yet.

The next have could be about

  1. writing small functions that do one thing well to clean the data
  2. adding multiple files to the workflow (we can use the data in activity-1/data/ *.json (there are two json files there that are smaller-ish)

@lwasser
Copy link
Member Author

lwasser commented Oct 24, 2024

Ok this is in a good enough state i think for an intro to pseudocode. Id still like to flesh it out more and i left a TODO where i started to do so. i'll open an issue about it


Of course, the pseudocode above is just the beginning. For each of the steps above, you need to flesh out how you can accomplish each task.
:::{todo}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Next steps for the pseudocode lesson:

  • flesh out the pseudocode further - have them turn a few steps into functions
  • Potentially??? have them add multiple files which will make them think more about functions and processing steps

@lwasser lwasser merged commit 677cd1a into pyOpenSci:main Oct 24, 2024
4 checks passed
@lwasser lwasser deleted the cleanup branch October 24, 2024 23:31
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

Successfully merging this pull request may close these issues.

1 participant