remove note to convert to sin_cos_tau instead sin_cos_pi master
authorJacob Lifshay <programmerjake@gmail.com>
Sat, 19 Jun 2021 00:09:56 +0000 (17:09 -0700)
committerJacob Lifshay <programmerjake@gmail.com>
Sat, 19 Jun 2021 00:09:56 +0000 (17:09 -0700)
In the process of converting the code, I discovered that
it would not remove any operations from sin_cos and that sin_cos_pi
would require an additional multiplication to implement, since what
the software implementation would really need instead of sin_cos_tau
would be for the input to be expressed as a integer number of right-
angles (how many multiples of 90 degrees is the input angle).

src/algorithms/trig_pi.rs

index 834fd9636b1a972a350ebd2cf52bec32fca7829e..5be0ad65b017a389823488c1050a1b22eeb51293 100644 (file)
@@ -3,9 +3,6 @@ use crate::{
     traits::{Compare, Context, ConvertFrom, ConvertTo, Float, Make, Select},
 };
 
-// TODO: change `sin`/`cos`/`sin_pi`/`cos_pi` implementations to be
-// in terms of `sin_cos_tau` aka. `(sin(2 * pi * x), cos(2 * pi * x))`
-
 mod consts {
     #![allow(clippy::excessive_precision)]
     #![allow(dead_code)]