Skip to content

Conversation

@Zheng-Bicheng
Copy link
Contributor

@Zheng-Bicheng Zheng-Bicheng commented Mar 2, 2023

@Zheng-Bicheng
Copy link
Contributor Author

@Liliyaw

@Zheng-Bicheng
Copy link
Contributor Author

运行后将输出图片的方向以及置信度

Snipaste_2023-03-02_16-21-42

@Zheng-Bicheng Zheng-Bicheng changed the title Update PPOCR-CLS Example 【Hackathon 215】Update PPOCR-CLS Example Mar 16, 2023
@Liliyaw
Copy link
Contributor

Liliyaw commented Mar 22, 2023

@Zheng-Bicheng 这个PR的一些修改意见如下哈~PR2的关注点可能也会类似,所以也可以基于以下原理先调整一版本。

  • 根目录
    修改根目录名称
    OCR-example
    |_ Text-recognition-example
    |_ Text-angle-classification-example

  • 图片目录

    • 修改目录名 img -> imgs
    • 添加图片来源说明的 README.md, 参考 imgs_words_en 的 README.md
  • Src 目录
    为了和后续单独PaddleClas的用例区分,所有单独用cls表示的都要修改成 angle_cls 或 textangle_cls

    • line 22 tvmgen_cls.h -> tvmgen_angle_cls.h or tvmgen_textangle_cls.h
    • line 33 ocr cls -> text angle classification
    • line 34 tvmgen_cls_outputs cls_outputs -> tvmgen_angle_cls_outputs angle_cls_outputs
    • line 37 tvmgen_cls_inputs cls_inputs -> tvmgen_angle_cls_inputs angle_cls_inputs
    • line 41 tvmgen_cls_run(&cls_inputs, &cls_outputs); -> tvmgen_angle_cls_run(&angle_cls_inputs, &angle_cls_outputs);
    • line 45&47 补充完整 conf 缩写的英文全称
  • README_CN.md 删掉
    因为我们这个仓库面向 Global 的, 所以目前暂时不需要中文的 README.md,但是你可以单独发给我,在后续提交PaddleOCR仓库的时候可以带着。辛苦了~

  • run_demo.sh

    • line 149 去掉中文备注 & line 152 保留原来的英文备注
    • line 150 ppcls -> angle_cls
    • line 164 ppcls -> angle_cls
    • line 167 cls -> angle_cls
    • line 169 cls.tar -> angle_cls.tar
    • line 170 cls.tar -> angle_cls.tar
    • line 172 img/word_10.png -> imgs/word_10.png
  • README.md

    • 修改题目为 -> "Running PaddleOCR text angle classification model on bare metal Arm(R) Cortex(R)-M55 CPU using Arm Virtual Hardware"
    • 添加一个 Model description 的部分说明模型的来源

其他的细节在你提交后, README.md 我会帮你修改~ 😊 Thanks for your patient waiting!

@Liliyaw Liliyaw merged commit 6e51670 into ArmDeveloperEcosystem:main Mar 23, 2023
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.

2 participants