feature/parallel_do #6730
feature/parallel_do #6730
Conversation
| */ | ||
| void ShrinkInLevel(size_t level, size_t elem_begin, size_t elem_end); | ||
|
|
||
| void MergeLoDTensor(const std::vector<const LoDTensor*>& lod_tensors, |
There was a problem hiding this comment.
Do we need to add comments about this function?
| t = &(var->Get<SelectedRows>().value()); | ||
| } else { | ||
| PADDLE_THROW("Variable type must be LoDTensor/SelectedRows."); | ||
| PADDLE_THROW("Variable type_id %s, expect LoDTensor/SelectedRows.", |
There was a problem hiding this comment.
expect LoDTensor/SelectedRows => expect LoDTensor/SelectedRows/Tensor?
There was a problem hiding this comment.
This is a leftover from debugging. Sorry for the confusion.
| t = var->GetMutable<SelectedRows>()->mutable_value(); | ||
| } else { | ||
| PADDLE_THROW("Variable type must be LoDTensor/SelectedRows."); | ||
| PADDLE_THROW("Variable type_id %s, expect LoDTensor/SelectedRows.", |
There was a problem hiding this comment.
expect LoDTensor/SelectedRows => expect LoDTensor/SelectedRows/Tensor?
There was a problem hiding this comment.
This is a leftover from debugging. Sorry for the confusion.
| return var->Get<SelectedRows>().GetCompleteDims(); | ||
| } else { | ||
| PADDLE_THROW("Variable type must be LoDTensor/SelectedRows."); | ||
| PADDLE_THROW("Variable %s type_id %s, expect LoDTensor/SelectedRows.", |
There was a problem hiding this comment.
This is a leftover from debugging. Sorry for the confusion.
| var->GetMutable<SelectedRows>()->set_height(dim[0]); | ||
| } else { | ||
| PADDLE_THROW("Variable type must be LoDTensor/SelectedRows."); | ||
| PADDLE_THROW("Variable %s type_id %s, expect LoDTensor/SelectedRows.", |
There was a problem hiding this comment.
This is a leftover from debugging. Sorry for the confusion.
paddle/framework/operator.cc
Outdated
| const Tensor* t = nullptr; | ||
| if (var->IsType<LoDTensor>()) { | ||
| t = &(var->Get<LoDTensor>()); | ||
| } else if (var->IsType<Tensor>()) { |
There was a problem hiding this comment.
A little confused about it: do the ParallelDo operator need it?
There was a problem hiding this comment.
This is a leftover from debugging. Sorry for the confusion.
paddle/operators/parallel_do_op.cc
Outdated
| distributed under the License is distributed on an "AS IS" BASIS, | ||
| WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| See the License for the specific language governing permissions and | ||
| limitations under the License. */ |
There was a problem hiding this comment.
The license format is not correct, please refer to #7022
4 participants
fix #6637
depends on: #6728