-
Notifications
You must be signed in to change notification settings - Fork 3.5k
[QNN-EP] Enable verbose and artifacts saving in onnxruntime_provider_test.exe #26396
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
base: main
Are you sure you want to change the base?
[QNN-EP] Enable verbose and artifacts saving in onnxruntime_provider_test.exe #26396
Conversation
…test.exe
- The change allows users to better debug unit tests by adding the
following command flags:
- --dump_onnx: Dump input onnx model
- --dump_json: Dump json qnn graph with provider_option
dump_json_qnn_graph
- --dump_dlc: Dump dlc with provider_option qnn_ir_backend_path
- --verbose: Use the log level ORT_LOGGING_LEVEL_VERBOSE
- Save onnx, dlc, json to the directory named with
<TestSuite>_<Testname>
- Skip datatypes check on QnnIr.dll to prevent generated dlc from
partitioned
|
Hi @edgchen1, could you take a look at this PR for QNN-EP's test cases and let me know if everything looks okay? |
| auto& logging_manager = DefaultLoggingManager(); | ||
| logging_manager.SetDefaultLoggerSeverity(log_severity); | ||
| if (QNNTestEnvironment::GetInstance().verbose()) { | ||
| logging_manager.RemoveSink(logging::SinkType::EtwSink); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
out of curiosity, why is the ETW log sink removed?
| if (QNNTestEnvironment::GetInstance().dump_json()) { | ||
| provider_options["dump_json_qnn_graph"] = "1"; | ||
| provider_options["json_qnn_graph_dir"] = output_dir.string(); | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: some of the test setup code is duplicated a few times. would it be worth refactoring it into helper functions?
|
/azp run Linux QNN CI Pipeline,Windows ARM64 QNN CI Pipeline |
|
Azure Pipelines successfully started running 2 pipeline(s). |
Description
QNN_DUMP_ONNX: Dump input onnx modelQNN_DUMP_JSON: Dump json qnn graph with provider_optiondump_json_qnn_graphQNN_DUMP_DLC: Dump dlc with provider_optionqnn_ir_backend_pathQNN_VERBOSE: Use the log levelORT_LOGGING_LEVEL_VERBOSE<TestSuite>_<TestName>Motivation and Context