Fix some LGTM issues.
This commit is contained in:
parent
2f446b7161
commit
aef3994e5c
@ -10,6 +10,7 @@
|
||||
"docdash": "^1.2.0",
|
||||
"esbuild": "^0.14.11",
|
||||
"eslint": "^8.5.0",
|
||||
"eslint-plugin-unused-imports": "^2.0.0",
|
||||
"fantasticon": "^1.2.2",
|
||||
"http-server": "^14.0.0",
|
||||
"jsdoc": "^3.6.7",
|
||||
@ -61,6 +62,9 @@
|
||||
"ecmaVersion": 12,
|
||||
"sourceType": "module"
|
||||
},
|
||||
"plugins": [
|
||||
"unused-imports"
|
||||
],
|
||||
"rules": {
|
||||
"semi": "error",
|
||||
"no-var": "error",
|
||||
@ -76,7 +80,7 @@
|
||||
"comma-spacing": "error",
|
||||
"prefer-arrow-callback": "error",
|
||||
"quotes": [
|
||||
"error",
|
||||
"error",
|
||||
"single"
|
||||
],
|
||||
"block-scoped-var": "error",
|
||||
@ -84,7 +88,8 @@
|
||||
"no-extend-native": "error",
|
||||
"eqeqeq": "error",
|
||||
"no-unused-vars": "off",
|
||||
"no-use-before-define": "off"
|
||||
"no-use-before-define": "off",
|
||||
"unused-imports/no-unused-imports": "error"
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@ -97,7 +97,7 @@ export class OctreeNode
|
||||
new Coord3D (minX, minY, minZ),
|
||||
new Coord3D (minX + sizeX, minY + sizeY, minZ + sizeZ)
|
||||
);
|
||||
node.childNodes.push (new OctreeNode (box, node.level + 1, node.options));
|
||||
node.childNodes.push (new OctreeNode (box, node.level + 1));
|
||||
}
|
||||
|
||||
let min = this.boundingBox.min;
|
||||
@ -145,7 +145,7 @@ export class Octree
|
||||
this.options.maxTreeDepth = options.maxTreeDepth;
|
||||
}
|
||||
}
|
||||
this.rootNode = new OctreeNode (boundingBox, 0, this.options);
|
||||
this.rootNode = new OctreeNode (boundingBox, 0);
|
||||
}
|
||||
|
||||
AddPoint (point, data)
|
||||
|
||||
@ -1,4 +1,3 @@
|
||||
import { VertexColors } from 'three';
|
||||
import { Coord2D } from '../geometry/coord2d.js';
|
||||
import { Coord3D } from '../geometry/coord3d.js';
|
||||
import { Direction } from '../geometry/geometry.js';
|
||||
|
||||
@ -186,7 +186,7 @@ export class ImporterThreeBase extends ImporterBase
|
||||
}
|
||||
for (let i = 0; i < threeMesh.geometry.groups.length; i++) {
|
||||
let group = threeMesh.geometry.groups[i];
|
||||
let groupEnd = group.start;
|
||||
let groupEnd = null;
|
||||
if (group.count === Infinity) {
|
||||
groupEnd = mesh.TriangleCount ();
|
||||
} else {
|
||||
|
||||
@ -3,13 +3,13 @@ import { Direction } from '../geometry/geometry.js';
|
||||
import { ColorToHexString } from '../model/color.js';
|
||||
import { ShadingType } from '../threejs/threeutils.js';
|
||||
import { GetDomElementInnerDimensions } from './domutils.js';
|
||||
import { Camera, CameraIsEqual3D, Navigation } from './navigation.js';
|
||||
import { Camera, Navigation } from './navigation.js';
|
||||
import { ViewerExtraGeometry, ViewerGeometry } from './viewergeometry.js';
|
||||
|
||||
export function GetDefaultCamera (direction)
|
||||
{
|
||||
if (direction === Direction.X) {
|
||||
return new CameraIsEqual3D (
|
||||
return new Camera (
|
||||
new Coord3D (2.0, -3.0, 1.5),
|
||||
new Coord3D (0.0, 0.0, 0.0),
|
||||
new Coord3D (1.0, 0.0, 0.0)
|
||||
|
||||
@ -1,6 +1,6 @@
|
||||
import { Color, ColorToHexString } from '../engine/model/color.js';
|
||||
import { CreateObjectUrl } from '../engine/io/bufferutils.js';
|
||||
import { AddDiv, CreateDiv, AddDomElement, CreateDomElement, GetDomElementOuterWidth, SetDomElementOuterWidth } from '../engine/viewer/domutils.js';
|
||||
import { AddDiv, CreateDiv, AddDomElement, GetDomElementOuterWidth, SetDomElementOuterWidth } from '../engine/viewer/domutils.js';
|
||||
import { CreateVerticalSplitter } from './splitter.js';
|
||||
|
||||
export function GetNameOrDefault (originalName, defaultName)
|
||||
|
||||
@ -1,10 +1,7 @@
|
||||
import os
|
||||
import sys
|
||||
import re
|
||||
import shutil
|
||||
|
||||
from lib import tools_lib as Tools
|
||||
|
||||
pickrFileMap = [
|
||||
[os.path.join ('@simonwep', 'pickr', 'LICENSE'), os.path.join ('pickr.license.md')],
|
||||
[os.path.join ('@simonwep', 'pickr', 'dist', 'pickr.es5.min.js'), os.path.join ('pickr.es5.min.js')],
|
||||
|
||||
Loading…
Reference in New Issue
Block a user