Skip to content
This repository has been archived by the owner on Jun 25, 2019. It is now read-only.

atan using x/y instead of y/x #14

Closed
asalga opened this issue Jul 3, 2014 · 2 comments
Closed

atan using x/y instead of y/x #14

asalga opened this issue Jul 3, 2014 · 2 comments

Comments

@asalga
Copy link

asalga commented Jul 3, 2014

In chapter 3, oscillation, why is atan being passed in x/y, shouldn't it be y/x?

V1 ⇒ angle = atan(-4/3) = atan(-1.25) = -0.9272952 radians = -53 degrees
V2 ⇒ angle = atan(4/-3) = atan(-1.25) = -0.9272952 radians = -53 degrees

@shiffman
Copy link
Member

thank you, indeed this looks like a mistake (also nature-of-code/The-Nature-of-Code-archive#199). Will check and fix when I do another round of updates!

shiffman added a commit that referenced this issue Jun 23, 2016
@shiffman
Copy link
Member

This is now how it looks, I believe this is right. Let me know if not:

screen shot 2016-06-23 at 11 27 26 am

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

No branches or pull requests

2 participants