-
Notifications
You must be signed in to change notification settings - Fork 0
Separation of variables skill builders #178
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
Conversation
|
mzbush
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This skill builder has done a pretty good job translating the notes. There are three main changes that need to happen before the pull request can be merged.
- All of these changes should be moved down to the bottom of the page after the examples.
- The problems should use the
{solution}directive so that the viewer can click to see the solution. I would recommend looking at the skill builder problems incalculus/series-expansion.mdto get a better idea of how to use the{solution}directive. - There should be some words to help explain the process of solving these equations.
I've added some suggested changes in problem a to improve the presentation of the problem and make it more in line with how the skill builder problems are expected to look. These sorts of changes should be applied to b and c as well.
first-order-odes/separable.md
Outdated
| Note that here, we are being careful to denote the redefinition of the | ||
| integration constant ($c = e^{c_0}$). This detail may be glossed over at times. | ||
|
|
||
| Solve the following: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Solve the following: | |
| ## Skill builder problems | |
| Solve the following: |
This section needs a heading
first-order-odes/separable.md
Outdated
|
|
||
| (a) | ||
|
|
||
| $$ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| $$ |
$$ is used to wrap equations that are on the same line as normal text. It is not needed here.
first-order-odes/separable.md
Outdated
| \begin{alignat*}{3} | ||
| y'+(x+2)y^2 = 0 ; \quad y(1) = 1 \\ | ||
| \frac{dy}{dx} = -(x+2)y^2 \\ | ||
| \int\frac{dy}{y^2} = \int-(x+2)dx \\ | ||
| -\frac{1}{y} = -\left(\frac{x^2}{2} + 2x\right) + C \\ | ||
| y(1) = -1 = -\left(\frac{1}{2} + 2 \right) + C & \quad \to & \quad C = \frac{3}{2} \\ | ||
| \to \quad y = \frac{2}{x^2+4x-3} | ||
| \end{alignat*} | ||
| $$ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| \begin{alignat*}{3} | |
| y'+(x+2)y^2 = 0 ; \quad y(1) = 1 \\ | |
| \frac{dy}{dx} = -(x+2)y^2 \\ | |
| \int\frac{dy}{y^2} = \int-(x+2)dx \\ | |
| -\frac{1}{y} = -\left(\frac{x^2}{2} + 2x\right) + C \\ | |
| y(1) = -1 = -\left(\frac{1}{2} + 2 \right) + C & \quad \to & \quad C = \frac{3}{2} \\ | |
| \to \quad y = \frac{2}{x^2+4x-3} | |
| \end{alignat*} | |
| $$ | |
| ```{solution} | |
| \begin{align} | |
| \frac{dy}{dx} &= -(x+2)y^2 \\ | |
| \int\frac{dy}{y^2} &= \int-(x+2)dx \\ | |
| -\frac{1}{y} &= -\left(\frac{x^2}{2} + 2x\right) + C \\ | |
| \end{align} | |
| \begin{equation} | |
| y(1) = -1 = -\left(\frac{1}{2} + 2 \right) + C \quad \to \quad C = \frac{3}{2} | |
| \end{equation} | |
| \begin{equation} | |
| \to \quad y = \frac{2}{x^2+4x-3} | |
| \end{equation} | |
| ``` |
Solving the problem should all go inside a {solution} directive like other skill builder problems. This solution could also use some words to explain what is happening and what each group of expressions is accomplishing in order to get to the solution. In this instance, I have separated the integration, solving for the constant C, and the final answer into groups. There should be some text between these different groups.
|
@kaylee0307 please make sure to finalize this and request a review from me ASAP to get credit for it! |
mzbush
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pull request looks really so far. I have some suggestions to fix up a few minor things, and then once all of them are addressed I think it will be ready for merging.
first-order-odes/separable.md
Outdated
| \begin{equation} | ||
| y = \sqrt{ln(x^2-2x+e} | ||
| \end{equation} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should there be a ± here so it's consistent with problem b? Then after stating to choose the positive, the final equation would be restated without the ±.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I applied Michaela's suggestions and reformatted your code.
New content
Resolves #93
Checklist
credits.md.