Skip to content

Add paddlemodel #45

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

Closed
wants to merge 2 commits into from
Closed

Add paddlemodel #45

wants to merge 2 commits into from

Conversation

an1018
Copy link
Contributor

@an1018 an1018 commented Jun 8, 2021

Add PaddleDetection layout parser model and PaddleOCR model


* For PubLayNet models, we suggest using `mask_rcnn_X_101_32x8d_FPN_3x` model as it's trained on the whole training set, while others are only trained on the validation set (the size is only around 1/50). You could expect a 15% AP improvement using the `mask_rcnn_X_101_32x8d_FPN_3x` model.
* []()

Choose a reason for hiding this comment

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

这一行,没用的话就删掉吧

@@ -373,9 +373,9 @@
],
"metadata": {

Choose a reason for hiding this comment

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

这个文档恢复下吧,咱们应该没有涉及到改动这个文档的改动

@@ -38,10 +38,51 @@ As reported by many users, the installation of Detectron2 can be rather tricky o
- [@ivanpp](https://github.com/ivanpp) curates a detailed description for installing `Detectron2` on Windows: [Detectron2 walkthrough (Windows)](https://ivanpp.cc/detectron2-walkthrough-windows/#step3installdetectron2)
- `Detectron2` maintainers claim that they won't provide official support for Windows (see [1](https://github.com/facebookresearch/detectron2/issues/9#issuecomment-540974288) and [2](https://detectron2.readthedocs.io/en/latest/tutorials/install.html)), but Detectron2 is continuously built on windows with CircleCI (see [3](https://github.com/facebookresearch/detectron2/blob/master/INSTALL.md#common-installation-issues)). Hopefully this situation will be improved in the future.

## [Optional] Install PaddleDetection for Using Layout Models

Choose a reason for hiding this comment

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

Install PaddleDetection -> Install PaddlePaddle


### For Windows and Linux Users

If you would like to use PaddleDetection deep learning models for layout detection, you also need to install paddle on your computer. This could be done by running the following command:

Choose a reason for hiding this comment

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

PaddleDetection -> PaddlePaddle


## [Optional] Install OCR utils

Layout Parser also comes with supports for OCR functions. In order to use them, you need to install the OCR utils via:
Layout Parser also comes with supports for OCR functions. In order to use them, you need to install the PaddleOCR utils via:

Choose a reason for hiding this comment

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

PaddleOCR -> OCR

@@ -12,7 +12,7 @@
version = __version__,
author = "Zejiang Shen Ruochen Zhang",
license = "Apache-2.0",
url = "https://github.com/Layout-Parser/layout-parser",
url = "https://github.com/an1018/layout-parser",

Choose a reason for hiding this comment

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

这块恢复下吧,之后可以让作者帮忙编包

@lolipopshock lolipopshock mentioned this pull request Jun 8, 2021
@@ -0,0 +1,142 @@
## Install Requirements:
Copy link
Member

@lolipopshock lolipopshock Jun 8, 2021

Choose a reason for hiding this comment

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

Do you have external documentations for training the Paddle models - we can add a link to that. But this should not be included in the layout parser documentation.

@an1018 an1018 closed this Jun 9, 2021
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.

3 participants