Init commit for DynRNN understand sentiment #30

reyoung · 2017-12-22T06:22:24Z

Dynamic RNN 情感分类

	总C++时间 16.64s
	* WhileOp 前向时间 -- 3.63s
		* 构造Op时间 0.63s
		* 构造Tensor时间 0.08s
		* 析构Op时间 0.27s
		* Op运行时间 2.16s
			InferShape时间 0.83
				* CompareOp::InferShape 0.04s
				* MulOp::InferShape 0.36s
					* String构造 0.08s
				* Activation::InferShape 0.05s
				* ElemwiseOp::InferShape 0.23s
				* SumOp::InferShape 0.15s
	* WhileOp  反向时间 -- 8.90s
		* 构造Op时间 0.72s
		* 析构Op时间 0.44s

	* 析构子Scope时间 -- 1.72s

dzhwinter · 2018-01-04T09:14:22Z

fluid/understand_sentiment_by_dynrnn.py

+        rnn.output(hidden)
+
+    last = fluid.layers.sequence_pool(rnn(), 'last')
+    logit = fluid.layers.fc(input=last, size=2, act='softmax')


We obey the convention that split model out of data pipeline,
just like https://github.com/dzhwinter/benchmark/blob/master/fluid/resnet.py#L75 .

dzhwinter · 2018-01-04T09:23:07Z

fluid/understand_sentiment_by_dynrnn.py

+        cPickle.dump(word_dict, f, cPickle.HIGHEST_PROTOCOL)
+
+
+def cache_reader(reader, clean):


We do not have a unified way to accelerate our benchmark demos. I doubt about the reasonability of caching the dataset with CPickle when do benchmark.

dzhwinter · 2018-01-04T09:24:12Z

fluid/understand_sentiment_by_dynrnn.py

@@ -0,0 +1,193 @@
+import argparse


We have a old one understand_sentiment_dynamic_rnn.py, I think we can delete that one and replace with this one?

dzhwinter

We can merge this PR and refine the io part style same with other scripts

reyoung added 5 commits December 22, 2017 14:20

Init commit for DynRNN understand sentiment

c67269b

Update data

be5d6e1

Update code

f4ff54d

Fix clean flag

19f706c

Update

5820efd

dzhwinter reviewed Jan 4, 2018

View reviewed changes

dzhwinter approved these changes Jan 8, 2018

View reviewed changes

dzhwinter merged commit dde94f9 into dzhwinter:master Jan 8, 2018

dzhwinter mentioned this pull request Jan 8, 2018

Add stacked dynamic lstm model for fluid. #34

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Init commit for DynRNN understand sentiment #30

Init commit for DynRNN understand sentiment #30

reyoung commented Dec 22, 2017 •

edited

dzhwinter Jan 4, 2018

dzhwinter Jan 4, 2018

dzhwinter Jan 4, 2018

dzhwinter left a comment

		cPickle.dump(word_dict, f, cPickle.HIGHEST_PROTOCOL)


		def cache_reader(reader, clean):

Init commit for DynRNN understand sentiment #30

Init commit for DynRNN understand sentiment #30

Conversation

reyoung commented Dec 22, 2017 • edited

dzhwinter Jan 4, 2018

Choose a reason for hiding this comment

dzhwinter Jan 4, 2018

Choose a reason for hiding this comment

dzhwinter Jan 4, 2018

Choose a reason for hiding this comment

dzhwinter left a comment

Choose a reason for hiding this comment

reyoung commented Dec 22, 2017 •

edited