Skip to content

TR1 feedback on first draft #689

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

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

TR1 feedback on first draft #689

wants to merge 5 commits into from

Conversation

AbbyANoble
Copy link
Contributor

This is @AbbyANoble's feedback for @kathypinzon's TR1. (Note to other TBIL editors--- please don't accept changes or merge this PR! This is for Kathy!)

This is @AbbyANoble's feedback for @kathypinzon's TR1. (Note to other TBIL editors--- please don't accept changes or merge this PR! This is for Kathy!)
Copy link

github-actions bot commented Apr 2, 2025

Comment on lines 212 to 215
Degrees are not the only way to measure an angle. We can also describe the angle's measure by the amount of the circumference of the circle that the angle's rotation created.
</p>
<p>
Recall that the circumference of a circle is given by <m>C=2\pi r</m>, where <m>r</m> is the radius of the circle. FINISH THIS!
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is not fixed yet. Will get to it on the next commit!

Copy link

github-actions bot commented Apr 2, 2025

Copy link

github-actions bot commented Apr 2, 2025

Copy link

github-actions bot commented Apr 2, 2025

@AbbyANoble AbbyANoble marked this pull request as ready for review April 3, 2025 18:59
@AbbyANoble
Copy link
Contributor Author

This is ready now @kathypinzon! I tried to add in some stuff about how what radians are. I also added in an activity for them to develop the pi/180 and 180/pi conversions as well as some fluency at the end. Let me know what you think!

Copy link

github-actions bot commented Apr 3, 2025

@siwelwerd
Copy link
Contributor

@AbbyANoble Should we just go ahead and merge this in? I read through the latest version with your improvements and I think it is pretty good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants