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

修改数据增强导致的DSR报错 #10662

Merged
merged 2 commits into from
Aug 17, 2023
Merged

修改数据增强导致的DSR报错 #10662

merged 2 commits into from
Aug 17, 2023

Conversation

xu-peng-7
Copy link
Contributor

由于在修改了dbnet数据增强的个数及顺序后,会导致ocrv4新增加的dsr增强失效,因此将其从固定索引改为检索索引,可以避免增加或减少数据增强后的错误

@paddle-bot
Copy link

paddle-bot bot commented Aug 17, 2023

Thanks for your contribution!

@@ -32,4 +32,4 @@
description: Format files with ClangFormat
entry: bash .clang_format.hook -i
language: system
files: \.(c|cc|cxx|cpp|cu|h|hpp|hxx|cuh|proto)$
Copy link
Collaborator

Choose a reason for hiding this comment

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

为什么修改pre-commit呢

Copy link
Contributor Author

Choose a reason for hiding this comment

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

改错了,所以又commit了一次,重新复制了一份进去

Copy link
Collaborator

@tink2123 tink2123 Aug 17, 2023

Choose a reason for hiding this comment

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

git add 的时候可以去掉对这个文件的修改哈~ 没有大的影响,先合入了

Copy link
Collaborator

@tink2123 tink2123 left a comment

Choose a reason for hiding this comment

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

LGTM

@tink2123 tink2123 merged commit 8f010ec into PaddlePaddle:dygraph Aug 17, 2023
@xu-peng-7 xu-peng-7 deleted the dataset branch August 17, 2023 08:01
xu-peng-7 added a commit to xu-peng-7/PaddleOCR that referenced this pull request Aug 18, 2023
* 修改数据增强导致的DSR报错

* 错误修改回滚
LDOUBLEV pushed a commit that referenced this pull request Aug 20, 2023
* 修改数据增强导致的DSR报错

* 错误修改回滚
@iweirman
Copy link

iweirman commented Sep 7, 2023

@xu-peng-7 大佬好,这个文件进行文字识别v4训练的时候依旧会报错。看样子应该需要在这里区分下是rec模型还是det模型。

@xu-peng-7
Copy link
Contributor Author

@xu-peng-7 大佬好,这个文件进行文字识别v4训练的时候依旧会报错。看样子应该需要在这里区分下是rec模型还是det模型。

收到,确实只测了文字检测。没有测试文字识别。尽快修

tink2123 added a commit that referenced this pull request Sep 7, 2023
* Update PP-OCRv4_introduction.md

* Update PP-OCRv4_introduction.md (#10616)

* Update PP-OCRv4_introduction.md

* Update PP-OCRv4_introduction.md

* Update PP-OCRv4_introduction.md

* Update README.md

* Cherrypicking GH-10217 and GH-10216 to PaddlePaddle:Release/2.7 (#10655)

* Don't break overall processing on a bad image

* Add preprocessing common to OCR tasks
Add preprocessing to options

* Update requirements.txt (#10656)

added missing pyyaml library

* [TIPC]update xpu tipc script (#10658)

* fix-typo (#10642)

Co-authored-by: Dennis <[email protected]>
Co-authored-by: shiyutang <[email protected]>

* 修改数据增强导致的DSR报错 (#10662) (#10681)

* 修改数据增强导致的DSR报错

* 错误修改回滚

* Update algorithm_overview_en.md (#10670)

Fixed simple spelling errors.

* Implement recoginition method ParseQ

* Document update for new recognition method ParseQ

* add prediction for parseq

* Update rec_vit_parseq.yml

* Update rec_r31_sar.yml

* Update rec_r31_sar.yml

* Update rec_r50_fpn_srn.yml

* Update rec_vit_parseq.py

* Update rec_vit_parseq.yml

* Update rec_parseq_head.py

* Update rec_img_aug.py

* Update rec_vit_parseq.yml

* Update __init__.py

* Update predict_rec.py

* Update paddleocr.py

* Update requirements.txt

* Update utility.py

* Update utility.py

---------

Co-authored-by: xiaoting <[email protected]>
Co-authored-by: topduke <[email protected]>
Co-authored-by: dyning <[email protected]>
Co-authored-by: UserUnknownFactor <[email protected]>
Co-authored-by: itasli <[email protected]>
Co-authored-by: Kai Song <[email protected]>
Co-authored-by: dvorst <[email protected]>
Co-authored-by: Dennis <[email protected]>
Co-authored-by: shiyutang <[email protected]>
Co-authored-by: Dec20B <[email protected]>
Co-authored-by: ncoffman <[email protected]>
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.

4 participants