Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update ONNX conversion readme_ch.md #11030

Merged
merged 2 commits into from
Oct 12, 2023
Merged

Conversation

greyovo
Copy link

@greyovo greyovo commented Oct 2, 2023

Fixed command line argument path

修正了转换步骤里的命令参数

Fixed command line argument path
@paddle-bot
Copy link

paddle-bot bot commented Oct 2, 2023

Thanks for your contribution!

@CLAassistant
Copy link

CLAassistant commented Oct 2, 2023

CLA assistant check
All committers have signed the CLA.

@shiyutang
Copy link
Collaborator

please push your changes to dygraph branch then cherry-pick to release/2.7

@shiyutang
Copy link
Collaborator

shiyutang commented Oct 11, 2023

你好, 你只是修改了路径,为什么需要修改呢

@greyovo
Copy link
Author

greyovo commented Oct 11, 2023

你好, 你只是修改了路径,为什么需要修改呢

因为路径打错了,少了斜杠,模型的输入的文件夹名字也写错了,直接运行是会报错的。

Copy link
Collaborator

@shiyutang shiyutang left a comment

Choose a reason for hiding this comment

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

LGTM

@shiyutang shiyutang merged commit 9ed3e5a into PaddlePaddle:release/2.7 Oct 12, 2023
@shiyutang
Copy link
Collaborator

请提交对应修改到dygraph

jzhang533 pushed a commit to jzhang533/PaddleOCR that referenced this pull request Mar 28, 2024
* Update ONNX conversion readme_ch.md

Fixed command line argument path

* Update ONNX conversion readme.md
gknor-edrone pushed a commit to edrone/PaddleOCR that referenced this pull request Sep 9, 2024
* Update ONNX conversion readme_ch.md

Fixed command line argument path

* Update ONNX conversion readme.md
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 11, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants