neuralmagic / deepsparse

Sparsity-aware deep learning inference runtime for CPUs
https://neuralmagic.com/deepsparse/
Other
2.94k stars 169 forks source link

Better legacy failure warning #1570

Open ProExpertProg opened 5 months ago

ProExpertProg commented 5 months ago

In case someone fails to create the V2 pipeline, I think it's helpful to print the error.

dsikka commented 5 months ago

You can't have the word error in your warning

ProExpertProg commented 5 months ago

Oh, I see. What words are allowed? Can I say "failed with"?

dsikka commented 5 months ago

@ProExpertProg you're currently failing this test: https://github.com/neuralmagic/deepsparse/blob/e29211c95b75ff19e0bf0c65a26b6044ba51bf80/tests/test_pipeline_benchmark.py#L89

Which checks for error and fail