Aapo Kyrola 63297e1a1f RunNetOnce->RunNet (removes rnn_executor overhead) ...

Summary:
seq2seq/translate.py was running much slower on RNNExecutor. This was because RNNExecutor has significant init overhead (I have another diff to reduce, but not completely eliminate it), and translate was calling the decoder with RunNetOnce -- thus always recreating the net and the ops. Changhing this to RunNet() makes translate run faster than without executor. RunNet uses the net name and uses the already created net, while RunNetOnce passes the whole protobuffer.

Noticed similar bug in seq2seq ensemble bean model, which also calls CreateNet() but uses RunNetOnce() instead of RunNet().

Reviewed By: jhcross

Differential Revision: D6156566

fbshipit-source-id: a933453e36a0d8fd163d0584186fda427a680687

2017-10-25 22:06:02 -07:00

..

beam_search.py

disable rnn executor for beam search

2017-10-24 20:49:56 -07:00

seq2seq_beam_search_test.py

Re-license to Apache

2017-09-28 16:22:00 -07:00

seq2seq_model_helper_test.py

Re-license to Apache

2017-09-28 16:22:00 -07:00

seq2seq_model_helper.py

Re-license to Apache

2017-09-28 16:22:00 -07:00

seq2seq_util.py

Re-license to Apache

2017-09-28 16:22:00 -07:00

train.py

Revert D6026557: [caffe2][PR] Fix "No handlers could be found for logger"

2017-10-12 20:21:52 -07:00

translate.py

RunNetOnce->RunNet (removes rnn_executor overhead)

2017-10-25 22:06:02 -07:00