search

presnick / RunestoneServer

Interactive books for computer science
http://interactivepython.org
Other
26 stars 46 forks source link

Function review problem solutions #37

Closed aerenchyma closed 10 years ago

aerenchyma commented 10 years ago

Tabbed problems and solutions for the problems done in section this week.

presnick commented 10 years ago

Oops. Wish. I had seen this before I just did these myself. Please check my solutions and suggest a merge.

Sent from my iPhone

On Feb 13, 2014, at 4:11 PM, jczetta notifications@github.com wrote:

Tabbed problems and solutions for the problems done in section this week.

You can merge this Pull Request by running

git pull https://github.com/aerenchyma/runestone func_review_prbsolns Or view, comment on, or merge it at:

https://github.com/presnick/runestone/pull/37

Commit Summary

working on fxn review problem addition working on review qs/solns week6 func review problems, solns done fix solns, clarity more clarification on solns File Changes

M pip/source/Assignments/week6.rst (179) Patch Links:

https://github.com/presnick/runestone/pull/37.patch https://github.com/presnick/runestone/pull/37.diff — Reply to this email directly or view it on GitHub.

aerenchyma commented 10 years ago

Are they really the same thing? Mine was just the ones we did in section -- not the ones from class. I'm happy to change anything but the solutions from class look fine to me...

On Thu, Feb 13, 2014 at 6:08 PM, Paul Resnick notifications@github.comwrote:

Oops. Wish. I had seen this before I just did these myself. Please check my solutions and suggest a merge.

Sent from my iPhone

On Feb 13, 2014, at 4:11 PM, jczetta notifications@github.com wrote:

Tabbed problems and solutions for the problems done in section this week.

You can merge this Pull Request by running

git pull https://github.com/aerenchyma/runestone func_review_prbsolns Or view, comment on, or merge it at:

https://github.com/presnick/runestone/pull/37

Commit Summary

working on fxn review problem addition working on review qs/solns week6 func review problems, solns done fix solns, clarity more clarification on solns File Changes

M pip/source/Assignments/week6.rst (179) Patch Links:

https://github.com/presnick/runestone/pull/37.patch

https://github.com/presnick/runestone/pull/37.diff

Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHubhttps://github.com/presnick/runestone/pull/37#issuecomment-35037785 .

aerenchyma commented 10 years ago

Oh -- because you'd first pulled in the new changes. Yeah, I'll suggest a merge for these now.

On Thu, Feb 13, 2014 at 6:19 PM, Jackie Cohen jczetta@gmail.com wrote:

Are they really the same thing? Mine was just the ones we did in section -- not the ones from class. I'm happy to change anything but the solutions from class look fine to me...

On Thu, Feb 13, 2014 at 6:08 PM, Paul Resnick notifications@github.comwrote:

Oops. Wish. I had seen this before I just did these myself. Please check my solutions and suggest a merge.

Sent from my iPhone

On Feb 13, 2014, at 4:11 PM, jczetta notifications@github.com wrote:

Tabbed problems and solutions for the problems done in section this week.

You can merge this Pull Request by running

git pull https://github.com/aerenchyma/runestone func_review_prbsolns Or view, comment on, or merge it at:

https://github.com/presnick/runestone/pull/37

Commit Summary

working on fxn review problem addition working on review qs/solns week6 func review problems, solns done fix solns, clarity more clarification on solns File Changes

M pip/source/Assignments/week6.rst (179) Patch Links:

https://github.com/presnick/runestone/pull/37.patch

https://github.com/presnick/runestone/pull/37.diff

Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHubhttps://github.com/presnick/runestone/pull/37#issuecomment-35037785 .

presnick commented 10 years ago

Oh, I misread the comment. I will merge them and post.

Sent from my iPhone

On Feb 13, 2014, at 6:24 PM, jczetta notifications@github.com wrote:

Oh -- because you'd first pulled in the new changes. Yeah, I'll suggest a merge for these now.

On Thu, Feb 13, 2014 at 6:19 PM, Jackie Cohen jczetta@gmail.com wrote:

Are they really the same thing? Mine was just the ones we did in section -- not the ones from class. I'm happy to change anything but the solutions from class look fine to me...

On Thu, Feb 13, 2014 at 6:08 PM, Paul Resnick notifications@github.comwrote:

Oops. Wish. I had seen this before I just did these myself. Please check my solutions and suggest a merge.

Sent from my iPhone

On Feb 13, 2014, at 4:11 PM, jczetta notifications@github.com wrote:

Tabbed problems and solutions for the problems done in section this week.

You can merge this Pull Request by running

git pull https://github.com/aerenchyma/runestone func_review_prbsolns Or view, comment on, or merge it at:

https://github.com/presnick/runestone/pull/37

Commit Summary

working on fxn review problem addition working on review qs/solns week6 func review problems, solns done fix solns, clarity more clarification on solns File Changes

M pip/source/Assignments/week6.rst (179) Patch Links:

https://github.com/presnick/runestone/pull/37.patch

https://github.com/presnick/runestone/pull/37.diff

Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHubhttps://github.com/presnick/runestone/pull/37#issuecomment-35037785 .

— Reply to this email directly or view it on GitHub.

presnick commented 10 years ago

Superseded by later pull request. Canceling this one.