Inconsistency of Squeeze spec between Ort and ONNX

[xkszltl]

Describe the bug

Ort uses 0-based index for axes in Squeeze but looks like ONNX says it should be 1-based.
https://github.com/onnx/onnx/blob/master/docs/Operators.md#Squeeze

axes : list of ints
List of positive integers, indicate the dimensions to squeeze.

System information

• OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Win10
• ONNX Runtime installed from (source or binary): source
• ONNX Runtime version: master
• Python version: 3.7.2
• GCC/Compiler version (if compiling from source): VS2017

[jywu-msft]

it's a technical error in the description then. it should be 0-based index for axes.
The example in the spec https://github.com/onnx/onnx/blob/master/docs/Operators.md#examples-97
even uses a 0 axis

node = onnx.helper.make_node(
'Squeeze',
inputs=['x'],
outputs=['y'],
axes=[0],
)
x = np.random.randn(1, 3, 4, 5).astype(np.float32)
y = np.squeeze(x, axis=0)

expect(node, inputs=[x], outputs=[y],
name='test_squeeze')

Given onnx is an interchange format between frameworks and caffe2, tf, pytorch etc. use 0-based index, it wouldn't make sense for onnx to use 1-based.

[xkszltl]

Forward to ONNX repo: onnx/onnx#1755