Skip to content

Conversation

@annxingyuan
Copy link
Contributor

@annxingyuan annxingyuan commented Mar 7, 2019

This PR fixes tensorflow/tfjs#1329

This PR fixes tensorflow/tfjs#1342


This change is Reviewable

@annxingyuan annxingyuan requested review from dsmilkov and nsthorat March 7, 2019 22:35
@annxingyuan annxingyuan self-assigned this Mar 7, 2019
Copy link

@nsthorat nsthorat left a comment

Choose a reason for hiding this comment

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

Reviewed 3 of 3 files at r1.
Reviewable status: :shipit: complete! 1 of 1 approvals obtained (waiting on @annxingyuan, @dsmilkov, and @nsthorat)


body-pix/README.md, line 84 at r1 (raw file):

  <head>
    <!-- Load TensorFlow.js -->
    <script src="https://cdn.jsdelivr.net/npm/@tensorflow/tfjs@0.13.3"></script>

let's keep this as a pinned version -- it ensures that a copy paste is sure to work

@annxingyuan
Copy link
Contributor Author

@nsthorat Do you want the pinned version to be 0.13.3 and 0.0.4? What do you think about pinning 1.0.0 for tfjs and body-pix?

@nsthorat
Copy link

nsthorat commented Mar 8, 2019

Yeah, let's pin 1.0.0!

Copy link
Contributor

@dsmilkov dsmilkov left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 3 files at r1, 1 of 1 files at r2.
Reviewable status: :shipit: complete! 2 of 1 approvals obtained (waiting on @dsmilkov)

@annxingyuan annxingyuan merged commit 5115cf1 into master Mar 9, 2019
@annxingyuan annxingyuan deleted the update_docs branch March 9, 2019 14:02
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.

Toxicity classifier demo showing different result with figure in readme. Using Body-pix as standalone es5 bundle is not working.

3 participants