nrontsis / PILCO

Bayesian Reinforcement Learning in Tensorflow
MIT License
313 stars 84 forks source link

Performance issue in the definition of create_models, pilco/controllers.py(P1) #59

Open DLPerf opened 3 years ago

DLPerf commented 3 years ago

Hello, I found a performance issue in the definition of create_models, pilco/controllers.py, tf.ones will be calculated repeatedly during program execution, resulting in reduced efficiency. I think it should be created before the loop.

The same issue exists in tf.ones in line 31 and tf.ones in line 19.

Looking forward to your reply. Btw, I am very glad to create a PR to fix it if you are too busy.