Skip to content

Conversation

@mattsoulanille
Copy link
Member

@mattsoulanille mattsoulanille commented Sep 21, 2022

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.

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


This change is Reviewable

@mattsoulanille
Copy link
Member Author

mattsoulanille commented Sep 21, 2022

This is working locally but fails on BrowserStack. Possibly related to missing files.

@mattsoulanille mattsoulanille marked this pull request as ready for review September 22, 2022 00:12
@mattsoulanille
Copy link
Member Author

Copy link
Collaborator

@jinjingforever jinjingforever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Thank you Matt!

Copy link
Collaborator

@lina128 lina128 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you Matt! This is great!

Reviewable status: :shipit: complete! 2 of 1 approvals obtained

@mattsoulanille mattsoulanille merged commit eaf8f05 into tensorflow:master Sep 22, 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.

3 participants