feat: rename reverse methods to split

Author: targos

matrix.d.ts

@@ -166,8 +166,8 @@ declare module 'ml-matrix' {
     mmulStrassen(y: Matrix): Matrix;
     scaleRows(min?: number, max?: number): Matrix;
     scaleColumns(min?: number, max?: number): Matrix;
-    reverseRows(): this;
-    reverseColumns(): this;
+    flipRows(): this;
+    flipColumns(): this;
     kroneckerProduct(other: Matrix): Matrix;
     tensorProduct(other: Matrix): Matrix;
     transpose(): Matrix;

src/__tests__/matrix/flip.js

@@ -0,0 +1,15 @@
+import { Matrix } from '../..';
+
+test('flip rows', () => {
+  const matrix = new Matrix([[1, 2, 3], [4, 5, 6]]);
+  const result = matrix.flipRows();
+  expect(result).toBe(matrix);
+  expect(result.to2DArray()).toStrictEqual([[3, 2, 1], [6, 5, 4]]);
+});
+
+test('flip columns', () => {
+  const matrix = new Matrix([[1, 2, 3], [4, 5, 6]]);
+  const result = matrix.flipColumns();
+  expect(result).toBe(matrix);
+  expect(result.to2DArray()).toStrictEqual([[4, 5, 6], [1, 2, 3]]);
+});

src/__tests__/matrix/reverse.js

@@ -1363,7 +1363,7 @@ export default class AbstractMatrix {
     return newMatrix;
   }
 
-  reverseRows() {
+  flipRows() {
     const middle = Math.ceil(this.columns / 2);
     for (var i = 0; i < this.rows; i++) {
       for (var j = 0; j < middle; j++) {
@@ -1376,7 +1376,7 @@ export default class AbstractMatrix {
     return this;
   }
 
-  reverseColumns() {
+  flipColumns() {
     const middle = Math.ceil(this.rows / 2);
     for (var j = 0; j < this.columns; j++) {
       for (var i = 0; i < middle; i++) {