Skip to content

Added unit tests for matrix arithmetic #108

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

Merged
merged 9 commits into from
Aug 29, 2019
Prev Previous commit
Next Next commit
Fixed swiftlint violations
  • Loading branch information
regexident committed Aug 29, 2019
commit 489b81757b791db9e572774659c2a5e6519c1892
6 changes: 2 additions & 4 deletions Tests/SurgeTests/MatrixTests.swift
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,8 @@
@testable import Surge
import XCTest

// swiftlint:disable nesting type_body_length

class MatrixTests: XCTestCase {
let floatAccuracy: Float = 1e-8
let doubleAccuracy: Double = 1e-11
Expand Down Expand Up @@ -363,7 +365,6 @@ class MatrixTests: XCTestCase {
}
)

// FIXME: Replace with `**` operator:
XCTAssertEqual(pow(lhs, rhs), expected, accuracy: 1e-5)
}

Expand All @@ -383,7 +384,6 @@ class MatrixTests: XCTestCase {
}
)

// FIXME: Replace with `**` operator:
XCTAssertEqual(pow(lhs, rhs), expected, accuracy: 1e-8)
}

Expand Down Expand Up @@ -429,7 +429,6 @@ class MatrixTests: XCTestCase {
[4, 5, 6],
]

// FIXME: replace with row iterator:
let expected: Matrix<Scalar> = [
[6],
[15],
Expand All @@ -446,7 +445,6 @@ class MatrixTests: XCTestCase {
[4, 5, 6],
]

// FIXME: replace with column iterator:
let expected: Matrix<Scalar> = [
[5, 7, 9],
]
Expand Down