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

Debugging Cont. - Update Problem set to use new debugging exercise #207

Merged
merged 5 commits into from
Aug 19, 2024

Conversation

kelsey-steven-ada
Copy link
Contributor

@kelsey-steven-ada kelsey-steven-ada commented Aug 16, 2024

Asana Task

Problem Set Replit

  • It was mean to be part of the task linked below, but I got ahead of myself and updated the Replit's PROBLEM_SET_README.md file while working on the Learn ticket
  • The file ACTIVITY_README.md for the in-class prompting activity still needs to be updated, that work is part of a different task


### !end-challenge
3. If we use the debugger or print statements to look at the values of `result` and `dna_encoded1`, we'll see that the last base in a sequence and its count is not being added to the compressed sequence stored in `result`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should dna_encoded1 be DNA_ENCODED?

debugging-contd/problem-set-debugging-contd.md Outdated Show resolved Hide resolved
debugging-contd/problem-set-debugging-contd.md Outdated Show resolved Hide resolved
Comment on lines 264 to 266
- Add print statements that print out `stage` at the beginning and the end of `one_generation`.

- Add print statements inside of the for loops that print out `stage`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Since these reference Game of Life, we could update them to be like:

  • Add a print statement to the test just before the assert statement to see the value of result.

@kelsey-steven-ada kelsey-steven-ada merged commit 8806eec into main Aug 19, 2024
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.

2 participants