Skip to content

Fixed model export logging + moved test data directory#1626

Merged
risemeup1 merged 1 commit intoPaddlePaddle:developfrom
alex-dinh:develop
Mar 18, 2026
Merged

Fixed model export logging + moved test data directory#1626
risemeup1 merged 1 commit intoPaddlePaddle:developfrom
alex-dinh:develop

Conversation

@alex-dinh
Copy link
Copy Markdown
Contributor

This PR fixes the logging during model exporting jobs (paddle2onnx/mapper/exporter.cc), particularly for when a model is not supported yet. There were many lines of code that should have logged important information in case of model conversion failure, but actually printed nothing. This should help with debugging and expanding support for new models in the future.

Output for an unsupported model before this change:

I0115 21:06:27.243992 77515 program_interpreter.cc:243] New Executor is Running.
[Paddle2ONNX] Start parsing the Paddle model file...
[ERROR][Paddle2ONNX] Due to the unsupported operators, the conversion is aborted.

Example output for an unsupported model after this change:

I0115 21:04:24.957676 76988 program_interpreter.cc:243] New Executor is Running.
[Paddle2ONNX] Start parsing the Paddle model file...
[Paddle2ONNX] Unsupported tensor operators were found:
- broadcast_tensors
[ERROR][Paddle2ONNX] Due to the unsupported operators, the conversion is aborted.

Additionally, this PR also changes the test data directory. Prior to this change, running tests locally creates many folders and files in the root directory, resulting in a lot of clutter. Instead, tests now create temp files under <project root>/test_data/

@paddle-bot
Copy link
Copy Markdown

paddle-bot bot commented Jan 16, 2026

Thanks for your contribution!

@risemeup1 risemeup1 merged commit b2b6394 into PaddlePaddle:develop Mar 18, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants