Skip to content

Implement SinPi, CosPi and TanPi and inverses #53675

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.

Sign up for GitHub

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

Closed
huoyaoyuan wants to merge 15 commits into from
Closed

Implement SinPi, CosPi and TanPi and inverses #53675

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
f1bcae0
SinPi
huoyaoyuan Jun 3, 2021
dd97080
CosPi
huoyaoyuan Jun 3, 2021
8004559
TanPi
huoyaoyuan Jun 3, 2021
4e9532f
Test IEEE compliance of +0 and -0
huoyaoyuan Jun 4, 2021
d2c82a4
Rewrite SinPi and CosPi
huoyaoyuan Jun 4, 2021
9d2576a
Add real implementation for TanPi
huoyaoyuan Jun 4, 2021
792825f
Add correctness test
huoyaoyuan Jun 4, 2021
ced1e7e
Add AsinPi, AcosPi, AtanPi and Atan2Pi
huoyaoyuan Jun 4, 2021
e702e36
Use the term half-revolution to match IEEE spec
huoyaoyuan Jun 4, 2021
1a43b7a
Test AsinPi, AcosPi, AtanPi and Atan2Pi
huoyaoyuan Jun 4, 2021
a81b6a2
Convert Atan2Pi test to theory to observe failures
huoyaoyuan Jun 5, 2021
63ab183
Special case AcosPi and AtanPi
huoyaoyuan Jun 5, 2021
7b74167
Special cases for Atan2Pi
huoyaoyuan Jun 5, 2021
4b809ba
Special case SinPi and change Atan2Pi
huoyaoyuan Jun 5, 2021
24a4444
Fix unwanted indent change and comment
huoyaoyuan Jun 5, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add correctness test
  • Loading branch information
@huoyaoyuan
huoyaoyuan committed Jun 4, 2021
commit 792825f2c8a2ec2593a30c64c3a970f834ce5cf2
24 changes: 24 additions & 0 deletions src/libraries/System.Runtime.Extensions/tests/System/Math.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3290,5 +3290,29 @@ public static void TanPi_Float_Precision()
AssertEqual(0.0f, MathF.TanPi(0.6789f) + MathF.TanPi(-0.6789f), 0.0f);
AssertEqual(0.0f, MathF.TanPi(1.2345f) + MathF.TanPi(-1.2345f), 0.0f);
}

[Theory]
[InlineData(0.1234)]
[InlineData(0.6789)]
[InlineData(1.2345)]
[InlineData(1.9876)]
public static void TriPi_Double_Correctness(double x)
{
AssertEqual(Math.Sin(x * Math.PI), Math.SinPi(x), CrossPlatformMachineEpsilonForEstimates);
AssertEqual(Math.Cos(x * Math.PI), Math.CosPi(x), CrossPlatformMachineEpsilonForEstimates);
AssertEqual(Math.Tan(x * Math.PI), Math.TanPi(x), CrossPlatformMachineEpsilonForEstimates);
}

[Theory]
[InlineData(0.1234f)]
[InlineData(0.6789f)]
[InlineData(1.2345f)]
[InlineData(1.9876f)]
public static void TriPi_Float_Correctness(float x)
{
AssertEqual(MathF.Sin(x * MathF.PI), MathF.SinPi(x), (float)CrossPlatformMachineEpsilonForEstimates);
AssertEqual(MathF.Cos(x * MathF.PI), MathF.CosPi(x), (float)CrossPlatformMachineEpsilonForEstimates);
AssertEqual(MathF.Tan(x * MathF.PI), MathF.TanPi(x), (float)CrossPlatformMachineEpsilonForEstimates);
}
}
}