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

Solution to 1.3 Limits-Analytically-(LT3) #481

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

mlamich1
Copy link
Contributor

No description provided.

Copy link

🚀 Preview available 🚀

https://a0cfe1ac.tbil.pages.dev

@StevenClontz
Copy link
Contributor

Reminder @mlamich1 to mark this as "ready for review" if you are done working on this branch

@mlamich1 mlamich1 marked this pull request as ready for review December 18, 2024 15:39
@StevenClontz StevenClontz added the seeking-reviewer This PR needs a volunteer to review it label Dec 18, 2024
@StevenClontz StevenClontz self-requested a review December 18, 2024 20:20
@StevenClontz StevenClontz added under-review This PR has been assigned a reviewer and removed seeking-reviewer This PR needs a volunteer to review it labels Dec 18, 2024
Comment on lines +423 to +441
<task><me>\lim_{x\to-6 } \dfrac{ x^{2} - 6 \, x + 5 }{ x^{2} - 3 \, x - 18 }</me>
<answer>
<p>
<m>\dfrac{77}{36}</m>
</p>
</answer></task>
<task><me>\lim_{x\to-1 } \dfrac{ x^{2} - 1 }{ x^{2} + 3 \, x + 2 }</me>
<answer>
<p>
<m>-2</m>
</p>
</answer></task>
<task><me>\lim_{x\to5 } \dfrac{ x - 5 }{ \sqrt{x + 31} - 6 }</me>
<answer>
<p>
<m>12</m>
</p>
</answer>
</task>
Copy link
Contributor

Choose a reason for hiding this comment

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

This is not your mistake, but we have some tasks inside of a statement that need to be fixed to use this structure:

<activity>
  <introduction><p>Determine....</p></introduction>
  <task>
    <statement><p><me>...</me></p></statement>
    <answer><p><m>...</m></p></answer>
  </task>
  <task>
    <statement><p><me>...</me></p></statement>
    <answer><p><m>...</m></p></answer>
  </task>
  <task>
    <statement><p><me>...</me></p></statement>
    <answer><p><m>...</m></p></answer>
  </task>
</activity>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

How can I do then? Or, you have fixed it.

Copy link
Contributor

Choose a reason for hiding this comment

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

This needs to be done in your Codespace on the Solution-to-1.3-Limits-Analytically-(LT3) branch

Copy link
Contributor

@StevenClontz StevenClontz left a comment

Choose a reason for hiding this comment

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

In addition to my suggestions above, there are still several <answer>s that need to be moved outside of their <statement>s.

@StevenClontz StevenClontz added awaiting-author and removed under-review This PR has been assigned a reviewer labels Dec 18, 2024
@StevenClontz StevenClontz marked this pull request as draft December 18, 2024 20:26
@StevenClontz StevenClontz removed their assignment Dec 18, 2024
Copy link

🚀 Preview available 🚀

https://3c97e9d2.tbil.pages.dev

@StevenClontz StevenClontz changed the base branch from Solution--for-Sec-1.2-Limits-Nemerically-LT2 to main December 18, 2024 20:33
@StevenClontz
Copy link
Contributor

This branch was made incorrectly (you forgot to branch off of main instead of your Solution--for-Sec-1.2-Limits-Nemerically-LT2 branch used for another PR), so there may be trouble merging this directly when it's ready.

@StevenClontz StevenClontz self-requested a review December 19, 2024 18:27
@mlamich1 mlamich1 marked this pull request as ready for review December 19, 2024 18:27
@StevenClontz StevenClontz added the under-review This PR has been assigned a reviewer label Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-author calculus under-review This PR has been assigned a reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants