Correct value_info ordering to ensure deterministic graph proto serialization

intel / onnxruntime

ONNX Runtime: cross-platform, high performance scoring engine for ML models

MIT License

56 stars 22 forks source link

Correct value_info ordering to ensure deterministic graph proto serialization #390

Closed jatinwadhwa921 closed 2 months ago

jatinwadhwa921 commented 2 months ago

Ensure deterministic ordering of value_info in graph proto serialization by sorting NodeArgs lexicographically by name

sfatimar commented 2 months ago

Please schedule a discussion with Preetha and Mayuresh to see if these issues can be resolved directly in OpenVINO as still two models are not working fine

preetha-intel commented 2 months ago

LGTM

preetha-intel commented 2 months ago

Raised PR with MSFT https://github.com/microsoft/onnxruntime/pull/21166