replace Math.trunc with Math.floor, #126

phetsims · Mar 26, 2024 · 9d427ac · 9d427ac
1 parent e66b746
commit 9d427ac
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/js/common/model/BarMagnetFieldGrid.ts b/js/common/model/BarMagnetFieldGrid.ts
@@ -147,8 +147,8 @@ export default class BarMagnetFieldGrid {
     if ( this.contains( x, y ) ) {
 
       // compute array indices
-      let columnIndex = Math.trunc( x / this.spacing );
-      let rowIndex = Math.trunc( y / this.spacing );
+      let columnIndex = Math.floor( x / this.spacing );
+      let rowIndex = Math.floor( y / this.spacing );
 
       // If we're at one of the index maximums, then we're exactly on the outer edge of the grid.
       // Back up by 1 so that we'll have a bounding rectangle.

diff --git a/js/common/model/Coil.ts b/js/common/model/Coil.ts
@@ -401,7 +401,7 @@ export default class Coil extends PhetioObject {
         numberOfElectrons = ELECTRONS_IN_RIGHT_END;
       }
       else {
-        numberOfElectrons = Math.trunc( this.loopRadiusProperty.value / ELECTRON_SPACING );
+        numberOfElectrons = Math.floor( this.loopRadiusProperty.value / ELECTRON_SPACING );
       }
       assert && assert( Number.isInteger( numberOfElectrons ) && numberOfElectrons > 0,
         `invalid numberOfElectrons: ${numberOfElectrons}` );

diff --git a/js/common/model/PickupCoilSamplePointsStrategy.ts b/js/common/model/PickupCoilSamplePointsStrategy.ts
@@ -71,7 +71,7 @@ export class FixedSpacingSamplePointsStrategy extends PickupCoilSamplePointsStra
   }
 
   protected override createSamplePointsProtected( loopRadius: number ): Vector2[] {
-    const numberOfSamplePointsOnRadius = Math.trunc( loopRadius / this.ySpacing );
+    const numberOfSamplePointsOnRadius = Math.floor( loopRadius / this.ySpacing );
     return createSamplePoints( numberOfSamplePointsOnRadius, this.ySpacing );
   }
 }