Fix initialization of same_node_ in TreeEnsemble

[xadupre]

Description

For TreeEnsemble, onnxruntime tries to fuse multiple nodes BRANCH_EQ into one node BRANCH_MEMBER. When a tree only contains BRANCH_EQ nodes, the final tree could be a mix between BRANCH_EQ and BRANCH_MEMBER. To be more efficient, onnxruntime detects that all the nodes use the same rule and avoids checking that value for every node while getting the final leaf. This detection happened before the fusion into BRANCH_MEMBER. This PR detects that this check must be done again. This extra cost only happens when a tree only contains nodes BRANCH_EQ and should not be much. It only happens during the initialization.

Motivation and Context

Fixes issue #24636.

[github-actions]

You can commit the suggested changes from lintrunner.

[cbourjau]

Excellent! Thanks for looking into this so quickly!

[yuslepukhin]

T operator*(double val) { return has_score ? score * static_cast(val) : 0; }

Potentially important nit: these two operators must be const.

---

Refers to: onnxruntime/core/providers/cpu/ml/tree_ensemble_aggregator.h:53 in 9ef3585. [](commit_id = 9ef3585, deletion_comment = False)

[yuslepukhin]

size_t limit;

Suggest both i and fpos to be size_t, that would reduce the number of casts

---

Refers to: onnxruntime/core/providers/cpu/ml/tree_ensemble_common.h:149 in 9ef3585. [](commit_id = 9ef3585, deletion_comment = False)

[yuslepukhin]

[yuslepukhin]

[xadupre]

/azp run build_x64_release

[azure-pipelines]

No pipelines are associated with this pull request.