Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tf_path in the english model #20

Open
Porcupine96 opened this issue Nov 11, 2019 · 1 comment
Open

tf_path in the english model #20

Porcupine96 opened this issue Nov 11, 2019 · 1 comment

Comments

@Porcupine96
Copy link

Hello,

I wanted to ask for the purpose of adding tf_path = tf_path + "/variables/variables" in the english model. There's no analogous tf_path modification in the chinese version of the script. Removing this line seems to fix my issue with converting the model.

tf_path = tf_path + "/variables/variables"

If it's indeed a bug, I've created a PR with a fix. If not, sorry for the trouble.

Thank you :)

@lonePatient
Copy link
Owner

@Porcupine96 Adding tf_path = tf_path + "/variables/variables" is for converting the weight of th_hub to the weight of pytorch.

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

No branches or pull requests

2 participants