Skip to content

Commit

Permalink
[wpimath] Remove unused SimpleMatrixUtils class
Browse files Browse the repository at this point in the history
It looks like it was gradually replaced by JNI wrappers around Eigen.
  • Loading branch information
calcmogul committed Nov 30, 2023
1 parent 23dd591 commit 6ab8907
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 271 deletions.
246 changes: 0 additions & 246 deletions wpimath/src/main/java/edu/wpi/first/math/SimpleMatrixUtils.java

This file was deleted.

25 changes: 0 additions & 25 deletions wpimath/src/test/java/edu/wpi/first/math/StateSpaceUtilTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,6 @@
import edu.wpi.first.wpilibj.UtilityClassTest;
import java.util.ArrayList;
import java.util.List;
import org.ejml.dense.row.MatrixFeatures_DDRM;
import org.ejml.simple.SimpleMatrix;
import org.junit.jupiter.api.Test;

class StateSpaceUtilTest extends UtilityClassTest<StateSpaceUtil> {
Expand Down Expand Up @@ -154,29 +152,6 @@ void testMatrixExp() {
1E-8));
}

@Test
void testSimpleMatrixExp() {
SimpleMatrix matrix = SimpleMatrixUtils.eye(2);
var result = SimpleMatrixUtils.exp(matrix);

assertTrue(
MatrixFeatures_DDRM.isIdentical(
result.getDDRM(),
new SimpleMatrix(2, 2, true, new double[] {Math.E, 0, 0, Math.E}).getDDRM(),
1E-9));

matrix = new SimpleMatrix(2, 2, true, new double[] {1, 2, 3, 4});
result = SimpleMatrixUtils.exp(matrix.scale(0.01));

assertTrue(
MatrixFeatures_DDRM.isIdentical(
result.getDDRM(),
new SimpleMatrix(
2, 2, true, new double[] {1.01035625, 0.02050912, 0.03076368, 1.04111993})
.getDDRM(),
1E-8));
}

@Test
void testPoseToVector() {
Pose2d pose = new Pose2d(1, 2, new Rotation2d(3));
Expand Down

0 comments on commit 6ab8907

Please sign in to comment.