Skip to content
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

[tracking] HF CNN fp32 model tests #801

Open
12 of 30 tasks
jinchen62 opened this issue Aug 2, 2024 · 5 comments
Open
12 of 30 tasks

[tracking] HF CNN fp32 model tests #801

jinchen62 opened this issue Aug 2, 2024 · 5 comments

Comments

@jinchen62
Copy link
Contributor

jinchen62 commented Aug 2, 2024

Below is the list of HF CNN fp32 model issues for Full model list. Tests are imported in nod-ai/SHARK-TestSuite#313.

Status on 10/1:

items tests model-run onnx-import torch-mlir iree-compile inference
total-count 529 499 489 418 313 161

Steps:
Build torch-mlir and iree locally

cd SHARK-TestSuite/e2eshark/
python ./run.py  --tolerance 0.001 0.001 -c path/to/torch-mlir/build -i path/to/iree-build --cachedir path/to/cache -f onnx -g models --mode onnx --report -j 12 --torchtolinalg --testsfile model_list.txt

model-run

onnxruntime.capi.onnxruntime_pybind11_state.RuntimeException: [ONNXRuntimeError] : 6 : RUNTIME_EXCEPTION : Exception during initialization: /onnxruntime_src/onnxruntime/core/optimizer/initializer.cc:49 onnxruntime::Initializer::Initializer(const onnx::TensorProto&, const onnxruntime::Path&) [ONNXRuntimeError] : 1 : FAIL : GetFileLength for ./patch_embed.proj.weight failed:Invalid fd was supplied: -1

onnx-import

torch-mlir

iree-compile

inference

@AmosLewis
Copy link
Contributor

onnx.Conv in #566 has been fixed by disable quantized 1d conv. This should be a different conv error.

@AmosLewis
Copy link
Contributor

Better to add the failed op into #797

@AmosLewis
Copy link
Contributor

onnx.Batchnorm in #566 also has been fixed by bump iree. This should be a different error. Maybe you should check vivek's patch llvm/torch-mlir#3597

@vivekkhandelwal1
Copy link
Contributor

Hi @jinchen62, can you please update this issue, since it seems some of these have been fixed.

@jinchen62
Copy link
Contributor Author

@vivekkhandelwal1 Nothing was changed by Monday, will verify and update weekly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants