labels rendering performance improvement: create ImageBitmaps in worker #6445
sashankaryalpr.yml
on: pull_request
modified-files
33s
teams
0s
Matrix: test / test-python
build
/
changes
21s
build
/
build
2m 14s
e2e
/
test-e2e
1m 30s
lint
/
eslint
1m 2s
test
/
test-app
1m 8s
all-tests
0s
Annotations
21 errors and 11 warnings
|
packages/app/src/utils.test.ts:
app/packages/looker/src/util.ts#L461
ReferenceError: Worker is not defined
❯ Module.createWorker packages/looker/src/util.ts:461:5
❯ packages/looker/src/lookers/abstract.ts:61:5
❯ packages/looker/src/lookers/abstract.ts:60:25
❯ packages/looker/src/lookers/abstract.ts:66:1
|
|
packages/looker/src/worker/disk-overlay-decoder.test.ts > decodeOverlayOnDisk > should fetch and decode overlay when label has overlay path field:
app/packages/state/src/recoil/utils.ts#L17
Error: [vitest] No "determinePathType" export is defined on the "@fiftyone/utilities" mock. Did you forget to return it from "vi.mock"?
If you need to partially mock a module, you can use "importOriginal" helper inside:
vi.mock("@fiftyone/utilities", async (importOriginal) => {
const actual = await importOriginal()
return {
...actual,
// your mocked methods
}
})
❯ Module.getSampleSrc packages/state/src/recoil/utils.ts:17:7
❯ Module.decodeOverlayOnDisk packages/looker/src/worker/disk-overlay-decoder.ts:61:27
❯ packages/looker/src/worker/disk-overlay-decoder.test.ts:78:11
|
|
packages/looker/src/worker/disk-overlay-decoder.test.ts > decodeOverlayOnDisk > should handle HEATMAP class:
app/packages/state/src/recoil/utils.ts#L17
Error: [vitest] No "determinePathType" export is defined on the "@fiftyone/utilities" mock. Did you forget to return it from "vi.mock"?
If you need to partially mock a module, you can use "importOriginal" helper inside:
vi.mock("@fiftyone/utilities", async (importOriginal) => {
const actual = await importOriginal()
return {
...actual,
// your mocked methods
}
})
❯ Module.getSampleSrc packages/state/src/recoil/utils.ts:17:7
❯ Module.decodeOverlayOnDisk packages/looker/src/worker/disk-overlay-decoder.ts:61:27
❯ packages/looker/src/worker/disk-overlay-decoder.test.ts:114:11
|
|
packages/looker/src/worker/disk-overlay-decoder.test.ts > decodeOverlayOnDisk > should handle DETECTIONS class and process detections recursively:
app/packages/state/src/recoil/utils.ts#L17
Error: [vitest] No "determinePathType" export is defined on the "@fiftyone/utilities" mock. Did you forget to return it from "vi.mock"?
If you need to partially mock a module, you can use "importOriginal" helper inside:
vi.mock("@fiftyone/utilities", async (importOriginal) => {
const actual = await importOriginal()
return {
...actual,
// your mocked methods
}
})
❯ Module.getSampleSrc packages/state/src/recoil/utils.ts:17:7
❯ decodeOverlayOnDisk packages/looker/src/worker/disk-overlay-decoder.ts:61:27
❯ Module.decodeOverlayOnDisk packages/looker/src/worker/disk-overlay-decoder.ts:35:9
❯ packages/looker/src/worker/disk-overlay-decoder.test.ts:166:11
|
|
packages/looker/src/worker/disk-overlay-decoder.test.ts > decodeOverlayOnDisk > should return early if fetch (with retry) fails:
app/packages/state/src/recoil/utils.ts#L17
Error: [vitest] No "determinePathType" export is defined on the "@fiftyone/utilities" mock. Did you forget to return it from "vi.mock"?
If you need to partially mock a module, you can use "importOriginal" helper inside:
vi.mock("@fiftyone/utilities", async (importOriginal) => {
const actual = await importOriginal()
return {
...actual,
// your mocked methods
}
})
❯ Module.getSampleSrc packages/state/src/recoil/utils.ts:17:7
❯ Module.decodeOverlayOnDisk packages/looker/src/worker/disk-overlay-decoder.ts:61:27
❯ packages/looker/src/worker/disk-overlay-decoder.test.ts:200:11
|
|
packages/playback/src/lib/utils.test.ts:
app/packages/looker/src/util.ts#L461
ReferenceError: Worker is not defined
❯ Module.createWorker packages/looker/src/util.ts:461:5
❯ packages/looker/src/lookers/abstract.ts:61:5
❯ packages/looker/src/lookers/abstract.ts:60:25
❯ packages/looker/src/lookers/abstract.ts:66:1
|
|
packages/state/src/recoil/aggregations.test.ts:
app/packages/looker/src/util.ts#L461
ReferenceError: Worker is not defined
❯ Module.createWorker packages/looker/src/util.ts:461:5
❯ packages/looker/src/lookers/abstract.ts:61:5
❯ packages/looker/src/lookers/abstract.ts:60:25
❯ packages/looker/src/lookers/abstract.ts:66:1
|
|
packages/state/src/recoil/dynamicGroups.test.ts:
app/packages/looker/src/util.ts#L461
ReferenceError: Worker is not defined
❯ Module.createWorker packages/looker/src/util.ts:461:5
❯ packages/looker/src/lookers/abstract.ts:61:5
❯ packages/looker/src/lookers/abstract.ts:60:25
❯ packages/looker/src/lookers/abstract.ts:66:1
|
|
packages/state/src/recoil/filters.test.ts:
app/packages/looker/src/util.ts#L461
ReferenceError: Worker is not defined
❯ Module.createWorker packages/looker/src/util.ts:461:5
❯ packages/looker/src/lookers/abstract.ts:61:5
❯ packages/looker/src/lookers/abstract.ts:60:25
❯ packages/looker/src/lookers/abstract.ts:66:1
|
|
packages/state/src/recoil/groups.test.ts:
app/packages/looker/src/util.ts#L461
ReferenceError: Worker is not defined
❯ Module.createWorker packages/looker/src/util.ts:461:5
❯ packages/looker/src/lookers/abstract.ts:61:5
❯ packages/looker/src/lookers/abstract.ts:60:25
❯ packages/looker/src/lookers/abstract.ts:66:1
|
|
test / test-python (ubuntu-latest-m, 3.9)
Canceling since a higher priority waiting request for 'Pull Request-refs/pull/5169/merge' exists
|
|
test / test-python (ubuntu-latest-m, 3.9)
The operation was canceled.
|
|
build / build
Canceling since a higher priority waiting request for 'Pull Request-refs/pull/5169/merge' exists
|
|
build / build
The operation was canceled.
|
|
test / test-python (ubuntu-latest-m, 3.11)
Canceling since a higher priority waiting request for 'Pull Request-refs/pull/5169/merge' exists
|
|
test / test-python (ubuntu-latest-m, 3.11)
The operation was canceled.
|
|
e2e / test-e2e
Canceling since a higher priority waiting request for 'Pull Request-refs/pull/5169/merge' exists
|
|
test / test-python (ubuntu-latest-m, 3.10)
Canceling since a higher priority waiting request for 'Pull Request-refs/pull/5169/merge' exists
|
|
e2e / test-e2e
The operation was canceled.
|
|
test / test-python (ubuntu-latest-m, 3.10)
The operation was canceled.
|
|
all-tests
Process completed with exit code 1.
|
|
lint / eslint:
app/packages/operators/src/CustomPanel.tsx#L40
React Hook useEffect has missing dependencies: 'panelId', 'panelName', 'setPanelCloseEffect', and 'trackEvent'. Either include them or remove the dependency array
|
|
lint / eslint:
app/packages/operators/src/CustomPanel.tsx#L105
React Hook useEffect has a missing dependency: 'dimensions'. Either include it or remove the dependency array
|
|
lint / eslint:
app/packages/operators/src/built-in-operators.ts#L815
'ctx' is defined but never used. Allowed unused args must match /^_/u
|
|
lint / eslint
'ctx' is defined but never used. Allowed unused args must match /^_/u
|
|
lint / eslint
'ctx' is defined but never used. Allowed unused args must match /^_/u
|
|
lint / eslint
'ctx' is defined but never used. Allowed unused args must match /^_/u
|
|
lint / eslint
'ctx' is defined but never used. Allowed unused args must match /^_/u
|
|
lint / eslint
'path' is never reassigned. Use 'const' instead
|
|
lint / eslint
'value' is never reassigned. Use 'const' instead
|
|
lint / eslint
'ctx' is defined but never used. Allowed unused args must match /^_/u
|
|
e2e / test-e2e
No files were found with the provided path: e2e-pw/playwright-report/. No artifacts will be uploaded.
|