Skip to content

Commit 42b8a0e

Browse files
committed
Minor fix
1 parent 9f2b667 commit 42b8a0e

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

src/tensorflow.cc

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -567,7 +567,6 @@ SetStringInputTensor(
567567
cudaStream_t stream, const char* host_policy_name)
568568
{
569569
bool cuda_copy = false;
570-
size_t element_idx = 0;
571570

572571
// For string data type, we always need to have the data on CPU so
573572
// that we can read string length and construct the string
@@ -582,11 +581,7 @@ SetStringInputTensor(
582581
&contiguous_buffer, stream, &cuda_copy);
583582
if (err != nullptr) {
584583
RESPOND_AND_SET_NULL_IF_ERROR(response, err);
585-
if (element_idx < request_element_cnt) {
586-
FillStringTensor(
587-
tensor, tensor_offset + element_idx,
588-
request_element_cnt - element_idx);
589-
}
584+
FillStringTensor(tensor, tensor_offset, request_element_cnt);
590585
free(contiguous_buffer);
591586
return cuda_copy;
592587
}
@@ -602,15 +597,16 @@ SetStringInputTensor(
602597
err = ValidateStringBuffer(
603598
content, content_byte_size, request_element_cnt, name, &str_list);
604599
// Set string values.
605-
for (; element_idx < str_list.size(); ++element_idx) {
600+
for (size_t element_idx = 0; element_idx < str_list.size(); ++element_idx) {
606601
const auto& [addr, len] = str_list[element_idx];
607602
TRITONTF_TensorSetString(tensor, tensor_offset + element_idx, addr, len);
608603
}
609604

605+
size_t element_cnt = str_list.size();
610606
if (err != nullptr) {
611607
RESPOND_AND_SET_NULL_IF_ERROR(response, err);
612608
FillStringTensor(
613-
tensor, tensor_offset + element_idx, request_element_cnt - element_idx);
609+
tensor, tensor_offset + element_cnt, request_element_cnt - element_cnt);
614610
}
615611
free(contiguous_buffer);
616612
return cuda_copy;

0 commit comments

Comments
 (0)