Skip to content

Field 'type' of 'value_info' is required but missing. #298

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
justinchuby opened this issue Jan 9, 2023 · 2 comments · Fixed by #307
Closed

Field 'type' of 'value_info' is required but missing. #298

justinchuby opened this issue Jan 9, 2023 · 2 comments · Fixed by #307
Assignees
Labels
broken ci bug Something isn't working

Comments

@justinchuby
Copy link
Collaborator

onnx checker in ONNX main raises

onnxscript/test/common/onnx_script_test_case.py:145: in run_converter_test
    onnx.checker.check_model(model)
../../anaconda3/envs/onnx/lib/python3.10/site-packages/onnx/checker.py:119: in check_model
    C.check_model(protobuf_string, full_check)
E   onnx.onnx_cpp2py_export.checker.ValidationError: Field 'type' of 'value_info' is required but missing.

The above exception was the direct cause of the following exception:
onnxscript/test/functions/onnxfns2_test.py:97: in test_onnxfns_reduce_log_sum_exp
    self.run_onnx_test(
onnxscript/test/common/onnx_script_test_case.py:251: in run_onnx_test
    self.run_converter_test(param, case.model)
onnxscript/test/common/onnx_script_test_case.py:154: in run_converter_test
    raise AssertionError("Verification of model failed.") from e
E   AssertionError: Verification of model failed.

for

@script()
def ReduceLogSumExp(data, axes: List[int], keepdims: int):
    t_axes = op.Constant(value_ints=axes)
    return op.Log(op.ReduceSum(op.Exp(data), t_axes, keepdims=keepdims))
@justinchuby justinchuby added the bug Something isn't working label Jan 9, 2023
@justinchuby
Copy link
Collaborator Author

@justinchuby
Copy link
Collaborator Author

justinchuby commented Jan 10, 2023

cc @gramalingam could you help take a look or point me to where I should fix?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
broken ci bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants