Skip to content

Commit 267b493

Browse files
authored
delete the supported domain version upper bounds (microsoft#23237)
### Description <!-- Describe your changes. --> This PR changes the range of ONNX versions supported by CANN graph inference to no upper limit (the previous version supports between 8 and 15), because the CANN version is further upgraded to support some developers' requirements for higher ONNX versions. ### Motivation and Context <!-- - Why is this change required? What problem does it solve? - If it fixes an open issue, please link to the issue here. -->
1 parent bb7f961 commit 267b493

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

onnxruntime/core/providers/cann/cann_graph.cc

+2-3
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,7 @@
1010
namespace onnxruntime {
1111
namespace cann {
1212

13-
static int lower_bound = 8; // Supported domain version lower bounds
14-
static int upper_bound = 15; // Supported domain version upper bounds
13+
static int lower_bound = 8; // Supported domain version lower bounds
1514

1615
std::once_flag flag;
1716

@@ -62,7 +61,7 @@ std::vector<NodeIndex> SupportONNXModel(const GraphViewer& graph_viewer) {
6261
for (const auto& index : graph_viewer.GetNodesInTopologicalOrder()) {
6362
const auto& node = graph_viewer.GetNode(index);
6463

65-
if (node->Domain() != kOnnxDomain || domain_version < lower_bound || domain_version > upper_bound ||
64+
if (node->Domain() != kOnnxDomain || domain_version < lower_bound ||
6665
!cann_supported_ops.count(node->OpType())) {
6766
unsupported_nodes.push_back(index);
6867
continue;

0 commit comments

Comments
 (0)