search

tensorflow / models

Models and examples built with TensorFlow
Other
76.99k stars 45.79k forks source link

loading a2 rather than a0 #11187

Open MustafaLotfi opened 5 months ago

MustafaLotfi commented 5 months ago

In the documentation that is provided for MoViNet tutorial, it says we will download MoViNet-A0-Stream. But In the code MoViNet-A2 is downloaded and used.

google-cla[bot] commented 5 months ago

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

review-notebook-app[bot] commented 5 months ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

laxmareddyp commented 5 months ago

Hi @Chen-Zhe @chaoyan1037 ,

PR having typo changes , Please approve in your free time.Thanks

LakshmiKalaKadali commented 1 week ago

Hi @Chen-Zhe, @chaoyan1037,

Could you please review the PR.

Thank You

MustafaLotfi commented 3 days ago

@chaoyan1037 In the previous cells, MoViNet-A2-base is loaded and used, Wouldn't it be a better approach to use a same version (A2) for stream model too?