PaddleOCR icon indicating copy to clipboard operation
PaddleOCR copied to clipboard

Published 20 hours ago • verified publisher icon PaddlePaddle

[WIP]merge uapi paddleocr

Open zhangyubo0722 opened this issue 9 months ago • 2 comments

zhangyubo0722 avatar May 08 '24 13:05 zhangyubo0722

Thanks for your contribution!

paddle-bot[bot] avatar May 08 '24 13:05 paddle-bot[bot]

please submit your work to main branch. and, what is uapi ?

I will convert it to draft, since it's wip.

jzhang533 avatar May 09 '24 05:05 jzhang533

please submit your work to main branch. and, what is uapi ?

I will convert it to draft, since it's wip.

This PR needs to synchronize the internal changes of pdx to the open source repo. After communicating with PaddleOCR developers, we plan to merge it into the 2.6.1 branch. We hope that this PR will be merged as soon as possible. Thank you.

zhangyubo0722 avatar May 15 '24 10:05 zhangyubo0722

Merging this patch to release/2.6.1 is an urgent request by @cuicheng01 from Baidu. Sure, we can merge it.

It would be helpful if PR author and stakeholders can explain the following:

  • what's the future plan for release/2.6.1 branch? if it will be used for internal fixes, probably we should rename it to a meaningful branch name.
  • is this change related to paddlex ?
  • what is uapi ?

jzhang533 avatar May 15 '24 11:05 jzhang533

Does this change require a cherry pick to the main branch?

GreatV avatar May 16 '24 05:05 GreatV

"release/2.6.1" is the stable branch of PaddleOCR for PaddleX dependencies. The new version of PaddleX will be open sourced at the end of May. This change is just a minor modification to the 2.6.1 branch that PaddleX previously relied on."uapi" is the core code of PaddleX. We don't think it's necessary to update the main branch. Of course, we can listen to @jzhang533 's advice.

cuicheng01 avatar May 24 '24 07:05 cuicheng01

"release/2.6.1" is the stable branch of PaddleOCR for PaddleX dependencies. The new version of PaddleX will be open sourced at the end of May. This change is just a minor modification to the 2.6.1 branch that PaddleX previously relied on."uapi" is the core code of PaddleX. We don't think it's necessary to update the main branch. Of course, we can listen to @jzhang533 's advice.

Please keep in mind that this project has transitioned to being fully community-driven. Seeking consensus and clear public communication in the community is crucial for this new model.

jzhang533 avatar May 24 '24 08:05 jzhang533

Please keep in mind that this project has transitioned to being fully community-driven. Seeking consensus and clear public communication in the community is crucial for this new model.

Alright, I understand. Thank you all for your remarkable work. As for the branch issue, we don't think the updates from this specific branch need to be cherry-pick to the main branch at this time. However, if anyone holds a different view, we can discuss it together. @GreatV

cuicheng01 avatar May 24 '24 09:05 cuicheng01

@cuicheng01 Got it.

GreatV avatar May 24 '24 09:05 GreatV