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

add define server #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Fieldhunter
Copy link

问题描述:

在实践5.2_best_practice.py的分布式代码当中,发现第74行的server未定义(书中也没有定义),导致运行时中出现server未定义的错误。

解决办法:

翻阅书本发现,在讲创建tensorflow集群时,有过server = tf.train.Server(cluster, job_name=FLAGS.job_name, task_index=FLAGS.task_index)的定义,而5.2_best_practice.py里并没有。自己尝试加上之后,PS作业无异常。

其他:

小小更正,也算不上什么大错误,书中的代码PS都是大写,而5.2_best_practice.py中PS为小写,如果不仔细看代码就会出错。

之前在issue中提到过这个问题,但没人回复我。所以我在此向您提出了PR。

add define server
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.

1 participant