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

fixed a bug; print at stage14 #3937

Merged
merged 1 commit into from
Feb 18, 2020
Merged

fixed a bug; print at stage14 #3937

merged 1 commit into from
Feb 18, 2020

Conversation

SunSki
Copy link
Contributor

@SunSki SunSki commented Feb 18, 2020

I made it available in python3

I made it available in python3
@@ -128,7 +128,7 @@ if [ $stage -le 14 ]; then
echo "$0: creating neural net configs using the xconfig parser";

num_targets=$(tree-info $treedir/tree |grep num-pdfs|awk '{print $2}')
learning_rate_factor=$(echo "print 0.5/$xent_regularize" | python)
learning_rate_factor=$(echo "print(0.5/$xent_regularize)" | python3)
Copy link
Contributor

Choose a reason for hiding this comment

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

this works the same in python2 and python3, I don't see why the change would be needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This error was happened in my case, so I changed like this

stage14

run_tdnn_aug_ernie_a.sh: creating neural net configs using the xconfig parser
tree-info exp/chain/tri5_7d_tree_aug/tree
File "", line 1
print 0.5/0.1
^
SyntaxError: Missing parentheses in call to 'print'. Did you mean print(0.5/0.1)?

@danpovey
Copy link
Contributor

danpovey commented Feb 18, 2020 via email

@danpovey danpovey merged commit cdf2d68 into kaldi-asr:master Feb 18, 2020
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.

2 participants