-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Open
Description
Hello! I've found a performance issue in your project:
tf.Session
being defined repeatedly leads to incremental overhead.
You can make your program more efficient by fixing this bug. Here is the Stack Overflow post to support it.
Below is detailed description about tf.Session being defined repeatedly:
- chapter_3/slim/datasets/download_and_convert_cifar10.py:
with tf.Session('') as sess
(here) is defined in the function_add_to_tfrecord
(here) which is repeatedly called in the loopfor i in range(_NUM_TRAIN_FILES)
(here). - chapter_2/cifar10_eval.py:
with tf.Session() as sess
(here) is defined in the functioneval_once
(here) which is repeatedly called in the loopwhile True
(here). - chapter_5/research/object_detection/eval_util.py:
sess = tf.Session(master, graph=tf.get_default_graph())
(here) is defined in the functionrun_checkpoint_once
(here) which is repeatedly called in the loopwhile True
(here). - chapter_5/research/slim/datasets/download_and_convert_cifar10.py:
with tf.Session('') as sess
(here) is defined in the function_add_to_tfrecord
(here) which is repeatedly called in the loopfor i in range(_NUM_TRAIN_FILES)
(here). - chapter_17/im2txt/evaluate.py:
with tf.Session() as sess
(here) is defined in the functionrun_once
(here) which is repeatedly called in the loopwhile True
(here). - chapter_9/server/serve.py:
sess = tf.Session(graph=graph)
(here) is repeatedly called in the loopfor name in os.listdir(a.local_models_dir)
(here). - chapter_10/delete_broken_img.py:
sess = tf.Session()
(here) is repeatedly called in the loopfor i, img_path in enumerate(all_pic_list)
(here). - chapter_10/pix2pix-tensorflow/server/serve.py:
sess = tf.Session(graph=graph)
(here) is repeatedly called in the loopfor name in os.listdir(a.local_models_dir)
(here).
tf.Session
being defined repeatedly could lead to incremental overhead. If you define tf.Session
out of the loop and pass tf.Session
as a parameter to the loop, your program would be much more efficient.
Looking forward to your reply. Btw, I am very glad to create a PR to fix it if you are too busy.
Metadata
Metadata
Assignees
Labels
No labels