Skip to content

Commit 8de1cfc

Browse files
committed
[tmva][sofie] Remove special case handling bool outputs
Since we use now for boolean tensors a std::vector<uint8_t> it is not needed to have a special treatment when the output ttype of the operator is a boolean (e.g. in Comparison)
1 parent 15563e2 commit 8de1cfc

File tree

1 file changed

+0
-10
lines changed

1 file changed

+0
-10
lines changed

tmva/sofie/inc/TMVA/ROperator_Comparision.hxx

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@ template<typename T, EComparisionOperator Op>
5656
class ROperator_Comparision final : public ROperator{
5757
private:
5858

59-
bool fIsModelOutput = false;
6059
std::string fNX1;
6160
std::string fNX2;
6261
std::string fNY;
@@ -283,12 +282,6 @@ public:
283282
model.PrintIntermediateTensors();
284283
}
285284
}
286-
287-
// check if this is not output operators to add a specific line for definining the tensor_xxx variable
288-
const auto & outputTensorNames = model.GetOutputTensorNames();
289-
fIsModelOutput = false;
290-
if (std::find(outputTensorNames.begin(), outputTensorNames.end(), fNY) != outputTensorNames.end())
291-
fIsModelOutput = true;
292285
}
293286

294287
std::string Generate(std::string opName) override {
@@ -374,9 +367,6 @@ public:
374367
out << "}\n";
375368
}
376369

377-
// since output is a boolean need to add the tensor_xxx variable since it is not defined as a pointer to a boolean std::vector
378-
if (!fIsModelOutput)
379-
out << SP << "const std::vector<std::uint8_t> & tensor_" << fNY << " = fTensor_" << fNY << ";\n";
380370

381371
return out.str();
382372
}

0 commit comments

Comments
 (0)