Skip to content

Conversation

@dydavidkim
Copy link

To see the logs from the Cloud Build CI, please join either our discussion or announcement mailing list.

xhcao and others added 5 commits September 21, 2022 17:25
WebGL backend is supposed to support 5D or 6D inputs for gatherND op. However, WebGL backend throws an error for such inputs because of the following codes in gatherND. Take an input tensor with (1,48,48,17,2) shape for example:

    ivec5 paramsShape = ivec5(1,48,48,17,2);
    ...
    paramsShape[j];

This is problematic because ivec5 is a strcuture defined by us, which could be accessed through [] and could be accessed only through .x/.y/.z/.w/.u.

This PR fixes this problem by applying paramsShape's value directly into the code, instead of using an intermediate ivec5.
Add a new test //tfjs-tflite/src/worker_test to test running tflite in a webworker. The test runs a simple model that just adds the two inputs together.
* Rescaling Preprocessing Layer
Co-authored-by:
David Kim (@koyykdy) <[email protected]>
Brian Zheng (@Brianzheng123) <[email protected]>

* PR issues resolved

* linting and PR issues resolved
Co-authored-by: Adam Lang (@AdamLang96) <[email protected]>
Co-authored-by: (@Brianzheng123) <[email protected]>

Co-authored-by: David Don Young Kim <[email protected]>
Co-authored-by: David Don Young Kim <[email protected]>
Prevent the webgl1 test env from being used in tfjs-layers unless it is explicitly requested (such as by running //tfjs-layers:tfjs-layers_webgl1_test).
@dydavidkim dydavidkim merged commit 417f593 into CodeSmithDSMLProjects:davidkim Sep 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants