Skip to content

Commit 59cd9f8

Browse files
made some new function names a bit shorter.
1 parent 0e2bee0 commit 59cd9f8

3 files changed

Lines changed: 18 additions & 18 deletions

File tree

packages/dev/base/lib/api/services/point.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -755,10 +755,10 @@ export class Point {
755755
* @param inputs Points, checkLastWithFirst flag, and tolerance
756756
* @returns Array of maximum fillet radii for each corner
757757
* @group fillet
758-
* @shortname max fillet radius half line
758+
* @shortname max fillets half line
759759
* @drawable false
760760
*/
761-
pointsMaxFilletsHalfLine(
761+
maxFilletsHalfLine(
762762
inputs: Inputs.Point.PointsMaxFilletsHalfLineDto
763763
): number[] {
764764
const { points, checkLastWithFirst = false, tolerance = 1e-7 } = inputs;
@@ -833,7 +833,7 @@ export class Point {
833833
safestPointsMaxFilletHalfLine(
834834
inputs: Inputs.Point.PointsMaxFilletsHalfLineDto
835835
): number {
836-
const allMaxRadii = this.pointsMaxFilletsHalfLine(inputs);
836+
const allMaxRadii = this.maxFilletsHalfLine(inputs);
837837

838838
if (allMaxRadii.length === 0) {
839839
// No corners, or fewer than 3 points. No fillet possible.

packages/dev/base/lib/api/services/polyline.test.ts

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -751,15 +751,15 @@ describe("Polyline unit tests", () => {
751751
it("should return an empty array for fewer than 3 points", () => {
752752
const points: Inputs.Base.Point3[] = [[0, 0, 0], [1, 1, 0]];
753753
const input: Inputs.Polyline.PolylineToleranceDto = { polyline: { points } };
754-
expect(polyline.polylineMaxFilletRadii(input)).toEqual([]);
754+
expect(polyline.maxFilletsHalfLine(input)).toEqual([]);
755755
});
756756

757757
it("should calculate fillet for the single corner of a 3-point open polyline", () => {
758758
const points: Inputs.Base.Point3[] = [[0, 0, 0], [5, 0, 0], [5, 3, 0]]; // 90 deg corner at [5,0,0]
759759
const input: Inputs.Polyline.PolylineToleranceDto = { polyline: { points, isClosed: false } };
760760
const expectedRadii = [1.5];
761761

762-
const result = polyline.polylineMaxFilletRadii(input);
762+
const result = polyline.maxFilletsHalfLine(input);
763763
uh.expectFloatArraysClose(result, expectedRadii, precision);
764764
});
765765

@@ -768,7 +768,7 @@ describe("Polyline unit tests", () => {
768768
const input: Inputs.Polyline.PolylineToleranceDto = { polyline: { points, isClosed: false } };
769769
const expectedRadii = [1.5, 1.5];
770770

771-
const result = polyline.polylineMaxFilletRadii(input);
771+
const result = polyline.maxFilletsHalfLine(input);
772772
uh.expectFloatArraysClose(result, expectedRadii, precision);
773773
});
774774

@@ -777,7 +777,7 @@ describe("Polyline unit tests", () => {
777777
const input: Inputs.Polyline.PolylineToleranceDto = { polyline: { points, isClosed: true } };
778778
const expectedRadii = [2/3, 1.5, 0.75];
779779

780-
const result = polyline.polylineMaxFilletRadii(input);
780+
const result = polyline.maxFilletsHalfLine(input);
781781
uh.expectFloatArraysClose(result, expectedRadii, precision);
782782
});
783783

@@ -786,7 +786,7 @@ describe("Polyline unit tests", () => {
786786
const input: Inputs.Polyline.PolylineToleranceDto = { polyline: { points, isClosed: true } };
787787
const expectedRadii = [1.5, 1.5, 1.5, 1.5];
788788

789-
const result = polyline.polylineMaxFilletRadii(input);
789+
const result = polyline.maxFilletsHalfLine(input);
790790
uh.expectFloatArraysClose(result, expectedRadii, precision);
791791
});
792792

@@ -795,7 +795,7 @@ describe("Polyline unit tests", () => {
795795
const input: Inputs.Polyline.PolylineToleranceDto = { polyline: { points, isClosed: false } };
796796
const expectedRadii = [0.0, 1.5];
797797

798-
const result = polyline.polylineMaxFilletRadii(input);
798+
const result = polyline.maxFilletsHalfLine(input);
799799
uh.expectFloatArraysClose(result, expectedRadii, precision);
800800
});
801801

@@ -805,7 +805,7 @@ describe("Polyline unit tests", () => {
805805
const input: Inputs.Polyline.PolylineToleranceDto = { polyline: { points, isClosed: false }, tolerance: customTolerance };
806806
const expectedRadii = [0.0];
807807

808-
const result = polyline.polylineMaxFilletRadii(input);
808+
const result = polyline.maxFilletsHalfLine(input);
809809
uh.expectFloatArraysClose(result, expectedRadii, precision);
810810
});
811811

@@ -816,26 +816,26 @@ describe("Polyline unit tests", () => {
816816
it("should return 0 if calculatePolylineMaxFillets returns an empty array", () => {
817817
const points: Inputs.Base.Point3[] = [[0, 0, 0], [1, 1, 0]];
818818
const input: Inputs.Polyline.PolylineToleranceDto = { polyline: { points } };
819-
expect(polyline.safestPolylineFilletRadius(input)).toBe(0);
819+
expect(polyline.safestFilletRadius(input)).toBe(0);
820820
});
821821

822822
it("should return the minimum of the calculated radii", () => {
823823
const points: Inputs.Base.Point3[] = [[0,0,0], [4,0,0], [0,3,0]];
824824
const input: Inputs.Polyline.PolylineToleranceDto = { polyline: { points, isClosed: true } };
825825
const expectedSafest = 2/3;
826-
expect(polyline.safestPolylineFilletRadius(input)).toBeCloseTo(expectedSafest, precision);
826+
expect(polyline.safestFilletRadius(input)).toBeCloseTo(expectedSafest, precision);
827827
});
828828

829829
it("should return 0 if any calculated radius is 0", () => {
830830
const points: Inputs.Base.Point3[] = [[0, 0, 0], [5, 0, 0], [10, 0, 0], [10, 3, 0]];
831831
const input: Inputs.Polyline.PolylineToleranceDto = { polyline: { points, isClosed: false } };
832-
expect(polyline.safestPolylineFilletRadius(input)).toBeCloseTo(0.0, precision);
832+
expect(polyline.safestFilletRadius(input)).toBeCloseTo(0.0, precision);
833833
});
834834

835835
it("should return the single radius if only one corner exists", () => {
836836
const points: Inputs.Base.Point3[] = [[0, 0, 0], [5, 0, 0], [5, 3, 0]];
837837
const input: Inputs.Polyline.PolylineToleranceDto = { polyline: { points, isClosed: false } };
838-
expect(polyline.safestPolylineFilletRadius(input)).toBeCloseTo(1.5, precision);
838+
expect(polyline.safestFilletRadius(input)).toBeCloseTo(1.5, precision);
839839
});
840840
});
841841

packages/dev/base/lib/api/services/polyline.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -467,10 +467,10 @@ export class Polyline {
467467
* @shortname polyline max fillet radii
468468
* @drawable false
469469
*/
470-
polylineMaxFilletRadii(
470+
maxFilletsHalfLine(
471471
inputs: Inputs.Polyline.PolylineToleranceDto
472472
): number[] {
473-
return this.point.pointsMaxFilletsHalfLine({
473+
return this.point.maxFilletsHalfLine({
474474
points: inputs.polyline.points,
475475
checkLastWithFirst: inputs.polyline.isClosed,
476476
tolerance: inputs.tolerance,
@@ -491,10 +491,10 @@ export class Polyline {
491491
* @shortname polyline safest fillet radius
492492
* @drawable false
493493
*/
494-
safestPolylineFilletRadius(
494+
safestFilletRadius(
495495
inputs: Inputs.Polyline.PolylineToleranceDto
496496
): number {
497-
const allMaxRadii = this.polylineMaxFilletRadii(inputs);
497+
const allMaxRadii = this.maxFilletsHalfLine(inputs);
498498

499499
if (allMaxRadii.length === 0) {
500500
// No corners, or fewer than 3 points. No fillet possible.

0 commit comments

Comments
 (0)