Skip to content

Commit f3bab3e

Browse files
Address PR comments: Remove redundant tests and rename others
Removed Float32HFFT and Float64HFFT as they were duplicates of ComplexFloat versions. Renamed Float32HFFT2/N and Float64HFFT2/N to ComplexFloat... to reflect input type. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent ff3c8d9 commit f3bab3e

File tree

1 file changed

+4
-30
lines changed

1 file changed

+4
-30
lines changed

test/TorchSharpTest/TestTorchTensor.cs

Lines changed: 4 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -6938,32 +6938,6 @@ public void Float64FFT()
69386938
Assert.Equal(ScalarType.ComplexFloat64, inverted.dtype);
69396939
}
69406940

6941-
[Fact]
6942-
[TestOf(nameof(fft.hfft))]
6943-
public void Float32HFFT()
6944-
{
6945-
var input = torch.arange(4, complex64);
6946-
var output = fft.hfft(input);
6947-
Assert.Equal(6, output.shape[0]);
6948-
Assert.Equal(ScalarType.Float32, output.dtype);
6949-
6950-
var inverted = fft.ihfft(output);
6951-
Assert.Equal(ScalarType.ComplexFloat32, inverted.dtype);
6952-
}
6953-
6954-
[Fact]
6955-
[TestOf(nameof(fft.hfft))]
6956-
public void Float64HFFT()
6957-
{
6958-
var input = torch.arange(4, complex128);
6959-
var output = fft.hfft(input);
6960-
Assert.Equal(6, output.shape[0]);
6961-
Assert.Equal(ScalarType.Float64, output.dtype);
6962-
6963-
var inverted = fft.ihfft(output);
6964-
Assert.Equal(ScalarType.ComplexFloat64, inverted.dtype);
6965-
}
6966-
69676941
[Fact]
69686942
[TestOf(nameof(fft.rfft))]
69696943
public void Float32RFFT()
@@ -7201,7 +7175,7 @@ public void Float64RFFTN()
72017175

72027176
[Fact]
72037177
[TestOf(nameof(fft.hfft2))]
7204-
public void Float32HFFT2()
7178+
public void ComplexFloat32HFFT2()
72057179
{
72067180
var input = torch.rand(new long[] { 5, 5, 5, 5 }, complex64);
72077181
var output = fft.hfft2(input);
@@ -7215,7 +7189,7 @@ public void Float32HFFT2()
72157189

72167190
[Fact]
72177191
[TestOf(nameof(fft.hfft2))]
7218-
public void Float64HFFT2()
7192+
public void ComplexFloat64HFFT2()
72197193
{
72207194
var input = torch.rand(new long[] { 5, 5, 5, 5 }, complex128);
72217195
var output = fft.hfft2(input);
@@ -7229,7 +7203,7 @@ public void Float64HFFT2()
72297203

72307204
[Fact]
72317205
[TestOf(nameof(fft.hfftn))]
7232-
public void Float32HFFTN()
7206+
public void ComplexFloat32HFFTN()
72337207
{
72347208
var input = torch.rand(new long[] { 5, 5, 5, 5 }, complex64);
72357209
var output = fft.hfftn(input);
@@ -7243,7 +7217,7 @@ public void Float32HFFTN()
72437217

72447218
[Fact(Skip = "Fails on all Release builds.")]
72457219
[TestOf(nameof(fft.hfftn))]
7246-
public void Float64HFFTN()
7220+
public void ComplexFloat64HFFTN()
72477221
{
72487222
if (!RuntimeInformation.IsOSPlatform(OSPlatform.Windows) && !RuntimeInformation.IsOSPlatform(OSPlatform.OSX)) {
72497223

0 commit comments

Comments
 (0)