-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/rnn to array to lod tensor #5411
Feature/rnn to array to lod tensor #5411
Conversation
LoD Rank Table stores the `level` of `lod` which is ordered by sequence length in descending order. It is useful when implement dynamic RNN and is shared by dynamic RNN memory, dynamic RNN slice input and dynamic RNN slice output operators.
…_array_to_lod_tensor
…om/reyoung/Paddle into feature/rnn_to_array_to_lod_tensor
…om/reyoung/Paddle into feature/rnn_to_array_to_lod_tensor
… feature/rnn_to_array_to_lod_tensor
…om/reyoung/Paddle into feature/rnn_to_array_to_lod_tensor
…om/reyoung/Paddle into feature/rnn_to_array_to_lod_tensor
…om/reyoung/Paddle into feature/rnn_to_array_to_lod_tensor
…om/reyoung/Paddle into feature/rnn_to_array_to_lod_tensor
…om/reyoung/Paddle into feature/rnn_to_array_to_lod_tensor
…ng/Paddle into feature/rnn_to_array_to_lod_tensor
@@ -45,7 +45,8 @@ void VarDescBind::SetLoDLevel(int32_t lod_level) { | |||
desc_.mutable_tensor_array()->set_lod_level(lod_level); | |||
break; | |||
default: | |||
PADDLE_THROW("Tensor type=%d does not support LoDLevel", desc_.type()); | |||
PADDLE_THROW("Tensor type=%d does not support LoDLevel", | |||
desc_.tensor_array().lod_level()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why substitute desc_.tensor_array().lod_level()
into type=%d
?
@@ -56,7 +57,8 @@ int32_t VarDescBind::GetLodLevel() const { | |||
case VarDesc::LOD_TENSOR_ARRAY: | |||
return desc_.tensor_array().lod_level(); | |||
default: | |||
PADDLE_THROW("Tensor type=%d does not support LoDLevel", desc_.type()); | |||
PADDLE_THROW("Tensor type=%d does not support LoDLevel", | |||
desc_.tensor_array().lod_level()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why substitute desc_.tensor_array().lod_level()
into type=%d
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Rnn to array to lod tensor
…om/reyoung/Paddle into feature/rnn_to_array_to_lod_tensor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.