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

[Feat] Add Paddle-to-ONNX model conversion script #2722

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

Bobholamovic
Copy link
Member

No description provided.

Copy link

paddle-bot bot commented Dec 24, 2024

Thanks for your contribution!

@@ -154,7 +154,7 @@ def main():
if len(sys.argv) == 1:
logging.warning("No arguments provided. Displaying help information:")
args_cfg().print_help()
return
sys.exit(1)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@TingquanGao 确认,这个算作异常行为,exit code设置为1。

@@ -172,7 +172,7 @@ def main():
if args.get_pipeline_config is not None:
interactive_get_pipeline(args.get_pipeline_config, args.save_path)
else:
return pipeline_predict(
pipeline_predict(
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修改后main() -> None,内部可以自由控制exit行为,这样也便于处理serve这种情况。

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

Successfully merging this pull request may close these issues.

1 participant