You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hello! I've found a performance issue in /translations/tfslim.py: with tf.Session() as sess(here) is repeatedly called in the loop for (net, shape, model_name) in models_list(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. Here is the Stack Overflow post to support it.
Looking forward to your reply. Btw, I am very glad to create a PR to fix it if you are too busy.
The text was updated successfully, but these errors were encountered:
Hello! I've found a performance issue in /translations/tfslim.py:
with tf.Session() as sess
(here) is repeatedly called in the loopfor (net, shape, model_name) in models_list
(here).tf.Session
being defined repeatedly could lead to incremental overhead. If you definetf.Session
out of the loop and passtf.Session
as a parameter to the loop, your program would be much more efficient. Here is the Stack Overflow post to support it.Looking forward to your reply. Btw, I am very glad to create a PR to fix it if you are too busy.
The text was updated successfully, but these errors were encountered: