Merge pull request #5119 from thunderstudying/dygraph

Create community_contribution_en.md

doc/doc_ch/code_and_doc.md

@@ -16,7 +16,7 @@ PaddleOCR的Python代码遵循 [PEP8规范](https://www.python.org/dev/peps/pep-
 
 - 空格 
 
-  - 空格应该加在逗号、分号、冒号前，而非他们的后面
+  - 空格应该加在逗号、分号、冒号后，而非他们的前面
 
     ```python
     # 正确：

doc/doc_en/code_and_doc.md

+  - Appendix
+
+    This appendix contains python, document specifications and Pull Request process. Please follow the relevant contents
+
+    - [Appendix 1：Python Code Specification](#Appendix1)
+
+    - [Appendix 2：Document Specification](#Appendix2)
+
+    - [Appendix 3：Pull Request Description](#Appendix3)
+
+    <a name="Appendix1"></a>
+
+    ## Appendix 1：Python Code Specification
+
+    The Python code of PaddleOCR follows [PEP8 Specification]( https://www.python.org/dev/peps/pep-0008/ ), some of the key concerns include the following
+
+    - Space 
+
+      - Spaces should be added after commas, semicolons, colons, not before them
+
+        ```python
+        # true:
+        print(x, y)
+        
+        # false:
+        print(x , y)
+        ```
+
+      - When specifying a keyword parameter or default parameter value in a function, do not use spaces on both sides of it
+
+        ```python
+        # true:
+        def complex(real, imag=0.0)
+        # false:
+        def complex(real, imag = 0.0)
+        ```
+
+    - comment
+
+      - Inline comments: inline comments are indicated by the` # `sign. Two spaces should be left between code and` # `, and one space should be left between` # `and comments, for example
+
+        ```python
+        x = x + 1  # Compensate for border
+        ```
+
+      - Functions and methods: The definition of each function should include the following:
+
+        - Function description: Utility, input and output of function
+
+        - Args: Name and description of each parameter
+        - Returns: The meaning and type of the return value
+
+        ```python
+        def fetch_bigtable_rows(big_table, keys, other_silly_variable=None):
+            """Fetches rows from a Bigtable.
+        
+            Retrieves rows pertaining to the given keys from the Table instance
+            represented by big_table.  Silly things may happen if
+            other_silly_variable is not None.
+        
+            Args:
+                big_table: An open Bigtable Table instance.
+                keys: A sequence of strings representing the key of each table row
+                    to fetch.
+                other_silly_variable: Another optional variable, that has a much
+                    longer name than the other args, and which does nothing.
+        
+            Returns:
+                A dict mapping keys to the corresponding table row data
+                fetched. Each row is represented as a tuple of strings. For
+                example:
+        
+                {'Serak': ('Rigel VII', 'Preparer'),
+                 'Zim': ('Irk', 'Invader'),
+                 'Lrrr': ('Omicron Persei 8', 'Emperor')}
+        
+                If a key from the keys argument is missing from the dictionary,
+                then that row was not found in the table.
+            """
+            pass
+        ```
+
+    <a name="Appendix2"></a>
+
+    ## Appendix 2: Document Specification
+
+    ### 2.1 Overall Description
+
+    - Document Location: If you add new features to your original Markdown file, please **Do not re-create** a new file. If you don't know where to add it, you can first PR the code and then ask the official in commit.
+
+    - New Markdown Document Name: Describe the content of the document in English, typically a combination of lowercase letters and underscores, such as `add_New_Algorithm.md`
+
+    - New Markdown Document Format: Catalog - Body - FAQ
+
+      > The directory generation method can use [this site](https://ecotrust-canada.github.io/markdown-toc/ ) Automatically extract directories after copying MD contents, and then add `<a name='XXXX'></a> before each heading of the MD file
+
+    - English and Chinese: Any changes or additions to the document need to be made in both Chinese and English documents.
+
+    ### 2.2 Format Specification
+
+    - Title format: The document title format follows the format of: Arabic decimal point combination-space-title (for example, `2.1 XXXX`, `2.XXXX`)
+
+    - Code block: Displays code in code block format that needs to be run, describing the meaning of command parameters before the code block. for example:
+
+      > Pipeline of detection + direction Classify + recognition: Vertical text can be recognized after set direction classifier parameters`--use_angle_cls true`.
+      >
+      > ```
+      > paddleocr --image_dir ./imgs/11.jpg --use_angle_cls true
+      > ```
+
+    - Variable Rrferences: If code variables or command parameters are referenced in line, they need to be represented in line code, for example, above `--use_angle_cls true` with one space in front and one space in back
+
+    - Uniform naming: e.g. PP-OCRv2, PP-OCR mobile, `paddleocr` whl package, PPOCRLabel, Paddle Lite, etc.
+
+    - Supplementary notes: Supplementary notes by reference format `>`.
+
+    - Picture: If a picture is added to the description document, specify the naming of the picture (describing its content) and add the picture under `doc/`.
+
+    - Title: Capitalize the first letter of each word in the title.
+
+    <a name="Appendix3"></a>
+
+    ## Appendix 3: Pull Request Description
+
+    ### 3.1 PaddleOCR Branch Description
+
+    PaddleOCR will maintain two branches in the future, one for each:
+
+    - release/x.x family branch: stable release version branch, also the default branch. PaddleOCR releases a new release branch based on feature updates and adapts to the release version of Paddle. As versions iterate, more and more release/x.x family branches are maintained by default with the latest version of the release branch.
+    - dygraph branch: For the development branch, adapts the dygraph version of the Paddle dynamic graph to primarily develop new functionality. If you need to redevelop, choose the dygraph branch. To ensure that the dygraph branch pulls out the release/x.x branch when needed, the code for the dygraph branch can only use the valid API in the latest release branch of Paddle. That is, if a new API has been developed in the Paddle dygraph branch but has not yet appeared in the release branch code, do not use it in Paddle OCR. In addition, performance optimization, parameter tuning, policy updates that do not involve API can be developed normally.
+
+    The historical branch of PaddleOCR will no longer be maintained in the future. These branches will continue to be maintained, considering that some of you may still be using them:
+
+    - Develop branch: This branch was used for the development and testing of static diagrams and is currently compatible with version >=1.7. If you have special needs, you can also use this branch to accommodate older versions of Paddle, but you won't update your code until you fix the bug.
+
+    PaddleOCR welcomes you to actively contribute code to repo. Here are some basic processes for contributing code.
+
+    ### 3.2 PaddleOCR Code Submission Process And Specification
+
+    > If you are familiar with Git use, you can jump directly to [Some Conventions For Submitting Code in 3.2.10](#Some_conventions_for_submitting_code)
+
+    #### 3.2.1 Create Your `Remote Repo`
+
+    - In PaddleOCR [GitHub Home]( https://github.com/PaddlePaddle/PaddleOCR ) Click the `Fork` button in the upper left corner to create a `remote repo`in your personal directory, such as ` https://github.com/ {your_name}/PaddleOCR`.
+
+    ![banner](../banner.png)
+
+    - Clone `Remote repo`
+
+    ```
+    # pull code of develop branch
+    git clone https://github.com/{your_name}/PaddleOCR.git -b dygraph
+    cd PaddleOCR
+    ```
+
+    > Clone failures are mostly due to network reasons, try again later or configure the proxy
+
+    #### 3.2.2 Login And Connect Using Token
+
+    Start by viewing the information for the current `remote repo`.
+
+    ```
+    git remote -v
+    # origin    https://github.com/{your_name}/PaddleOCR.git (fetch)
+    # origin    https://github.com/{your_name}/PaddleOCR.git (push)
+    ```
+
+    Only the information of the clone `remote repo`, i.e. the PaddleOCR under your username, is available. Due to the change in Github's login method, you need to reconfigure the `remote repo` address by means of a Token. The token is generated as follows:
+    
+    1. Find Personal Access Tokens: Click on your avatar in the upper right corner of the Github page and choose Settings --> Developer settings --> Personal access tokens,
+    
+    2. Click Generate new token: Fill in the token name in Note, such as 'paddle'. In Select scopes, select repo (required), admin:repo_hook, delete_repo, etc. You can check them according to your needs. Then click Generate token to generate the token, and finally copy the generated token.
+
+    Delete the original origin configuration
+   
+    ```
+    git remote rm origin
+    ```
+    
+    Change the remote branch to `https://oauth2:{token}@github.com/{your_name}/PaddleOCR.git`. For example, if the token value is 12345 and your user name is PPOCR, run the following command
+    
+    ```
+    git remote add origin https://oauth2:12345@github.com/PPOCR/PaddleOCR.git
+    ```
+    
+    This establishes a connection to our own `remote repo`. Next we create a remote host of the original PaddleOCR repo, named upstream.
+
+    ```
+    git remote add upstream https://github.com/PaddlePaddle/PaddleOCR.git
+    ```
+
+    Use `git remote -v` to view current `remote warehouse` information, output as follows, found to include two origin and two upstream of `remote repo` .
+
+    ```
+    origin    https://github.com/{your_name}/PaddleOCR.git (fetch)
+    origin    https://github.com/{your_name}/PaddleOCR.git (push)
+    upstream    https://github.com/PaddlePaddle/PaddleOCR.git (fetch)
+    upstream    https://github.com/PaddlePaddle/PaddleOCR.git (push)
+    ```
+
+    This is mainly to keep the local repository up to date when subsequent pull request (PR) submissions are made.
+
+    #### 3.2.3 Create Local Branch
+
+    First get the latest code of upstream, then create a new_branch branch based on the dygraph of the upstream repo (upstream).
+    
+    ```
+    git fetch upstream
+    git checkout -b new_branch upstream/dygraph
+    ```
+    
+    > If for a newly forked PaddleOCR project, the user's remote repo (origin) has the same branch updates as the upstream repository (upstream), you can also create a new local branch based on the default branch of the origin repo or a specified branch with the following command
+    >
+    > ```
+    > # Create new_branch branch on user remote repo (origin) based on develop branch
+    > git checkout -b new_branch origin/develop
+    > # Create new_branch branch based on upstream remote repo develop branch
+    > # If you need to create a new branch from upstream, 
+    > # you need to first use git fetch upstream to get upstream code
+    > git checkout -b new_branch upstream/develop
+    > ```
+
+    The final switch to the new branch is displayed with the following output information.
+
+    ```
+    Branch new_branch set up to track remote branch develop from upstream.
+    Switched to a new branch 'new_branch'
+    ```
+    
+    After switching branches, file changes can be made on this branch
+    
+    #### 3.2.4 Use Pre-Commit Hook
+
+    Paddle developers use the pre-commit tool to manage Git pre-submit hooks. It helps us format the source code (C++, Python) and automatically check for basic things (such as having only one EOL per file, not adding large files to Git) before committing it.
+
+    The pre-commit test is part of the unit test in Travis-CI. PR that does not satisfy the hook cannot be submitted to PaddleOCR. Install it first and run it in the current directory：
+
+    ```
+    pip install pre-commit
+    pre-commit install
+    ```
+
+     >  1. Paddle uses clang-format to adjust the C/C++ source code format. Make sure the `clang-format` version is above 3.8.
+     >
+     >  2. Yapf installed through pip install pre-commit is slightly different from conda install-c conda-forge pre-commit, and PaddleOCR developers use `pip install pre-commit`.
+
+    #### 3.2.5 Modify And Submit Code
+
+     If you make some changes on `README.Md ` on PaddleOCR, you can view the changed file through `git status`, and then add the changed file using `git add`。
+
+    ```
+    git status # View change files
+    git add README.md
+    pre-commit
+    ```
+
+    Repeat these steps until the pre-comit format check does not error. As shown below.
+
+    ![img](../precommit_pass.png)
+
+    Use the following command to complete the submission.
+
+    ```
+    git commit -m "your commit info"
+    ```
+
+    #### 3.2.6 Keep Local Repo Up To Date
+
+    Get the latest code for upstream and update the current branch. Here the upstream comes from section 2.2, `Connecting to a remote repo`.
+
+    ```
+    git fetch upstream
+    # If you want to commit to another branch, you need to pull code from another branch of upstream, here is develop
+    git pull upstream develop
+    ```
+
+    #### 3.2.7 Push To Remote Repo
+
+    ```
+    git push origin new_branch
+    ```
+
+    #### 3.2.7 Submit Pull Request
+
+    Click the new pull request to select the local branch and the target branch, as shown in the following figure. In the description of PR, fill in the functions completed by the PR. Next, wait for review, and if you need to modify something, update the corresponding branch in origin with the steps above.
+
+    ![banner](../pr.png)
+
+    #### 3.2.8 Sign CLA Agreement And Pass Unit Tests
+
+    - Signing the CLA When submitting a Pull Request to PaddlePaddle for the first time, you need to sign a CLA (Contributor License Agreement) agreement to ensure that your code can be incorporated as follows:
+
+      1. Please check the Check section in PR, find the license/cla, and click on the right detail to enter the CLA website
+
+      2. Click Sign in with GitHub to agree on the CLA website and when clicked, it will jump back to your Pull Request page
+
+    #### 3.2.9 Delete Branch
+
+    - Remove remote branch
+
+      After PR is merged into the main repo, we can delete the branch of the remote repofrom the PR page.
+      You can also use `git push origin:branch name` to delete remote branches, such as:
+
+  ```
+    git push origin :new_branch
+  ```
+
+- Delete local branch
+
+  ```
+      # Switch to the development branch, otherwise the current branch cannot be deleted
+      git checkout develop
+      
+      # Delete new_ Branch Branch
+      git branch -D new_branch
+  ```
+
+    <a name="Some_conventions_for_submitting_code"></a>
+
+    #### 3.2.10 Some Conventions For Submitting Code
+
+    In order for official maintainers to better focus on the code itself when reviewing it, please follow the following conventions each time you submit your code:
+
+    1）Please ensure that the unit tests in Travis-CI pass smoothly. If not, indicate that there is a problem with the submitted code, and the official maintainer generally does not review it.
+    
+    2）Before submitting a Pull Request.
+    
+    - Note the number of commits.
+
+      Reason: If you only modify one file and submit more than a dozen commits, each commit will only make a few modifications, which can be very confusing to the reviewer. The reviewer needs to look at each commit individually to see what changes have been made, and does not exclude the fact that changes between commits overlap each other.
+      
+      Suggestion: Keep as few commits as possible each time you submit, and supplement your last commit with git commit --amend. For multiple commits that have been Push to a remote warehouse, you can refer to [squash commits after push](https://stackoverflow.com/questions/5667884/how-to-squash-commits-in-git-after-they-have-been-pushed ).
+
+    - Note the name of each commit: it should reflect the content of the current commit, not be too arbitrary.
+
+
+    3） If you have solved a problem, add in the first comment box of the Pull Request:fix #issue_number，This will automatically close the corresponding Issue when the Pull Request is merged. Key words include:close, closes, closed, fix, fixes, fixed, resolve, resolves, resolved,please choose the right vocabulary. Detailed reference [Closing issues via commit messages](https://help.github.com/articles/closing-issues-via-commit-messages).
+    
+    In addition, in response to the reviewer's comments, you are requested to abide by the following conventions:
+    
+    1） Each review comment from an official maintainer would like a response, which would better enhance the contribution of the open source community.
+    
+    - If you agree to the review opinion and modify it accordingly, give a simple Done.
+    - If you disagree with the review, please give your own reasons for refuting.
+    
+    2）If there are many reviews:
+    
+    - Please give an overview of the changes.
+    - Please reply with `start a review', not directly. The reason is that each reply sends an e-mail message, which can cause a mail disaster.

doc/doc_en/community_contribution_en.md

+# COMMUNITY CONTRIBUTION
+
+Thank you for your support and interest in PaddleOCR. The goal of  PaddleOCR is to build a professional, harmonious and supportive open source community with developers. This document presents existing community contributions, explanations for various contributions, and new opportunities and processes to make the contribution process more efficient and clear.
+
+PaddleOCR wants to help any developer with a dream realize their vision and enjoy the joy of creating value through the power of AI.
+
+---
+
+<a href="https://github.com/PaddlePaddle/PaddleOCR/graphs/contributors">
+  <img src="https://contrib.rocks/image?repo=PaddlePaddle/PaddleOCR" />
+</a>
+
+> The picture above shows PaddleOCR's current Contributor, updated regularly
+
+## 1. COMMUNITY CONTRIBUTION 
+
+### 1.1 PaddleOCR BASED COMMUNITY PROJECT
+
+- 【The lastest】 [FastOCRLabel](https://gitee.com/BaoJianQiang/FastOCRLabel): Complete C# version annotation tool (@ [包建强](https://gitee.com/BaoJianQiang) )
+
+#### 1.1.1 UNIVERSAL TOOL
+
+- [DangoOCR offline version](https://github.com/PantsuDango/DangoOCR)：Universal desktop instant translation tool (@ [PantsuDango](https://github.com/PantsuDango))
+- [scr2txt](https://github.com/lstwzd/scr2txt)：Screenshot to Text tool (@ [lstwzd](https://github.com/lstwzd))
+- [AI Studio project](https://aistudio.baidu.com/aistudio/projectdetail/1054614?channelType=0&channel=0)：English video automatically generates subtitles( @ [叶月水狐](https://aistudio.baidu.com/aistudio/personalcenter/thirdview/322052))
+
+#### 1.1.2 VERTICAL SCENE TOOLS
+
+- [id_card_ocr](https://github.com/baseli/id_card_ocr)：Identification of copy of ID card(@ [baseli](https://github.com/baseli))
+- [Paddle_Table_Image_Reader](https://github.com/thunder95/Paddle_Table_Image_Reader): A data assistant that can read tables and pictures(@ [thunder95](https://github.com/thunder95]))
+
+#### 1.1.3 PRE AND POST PROCESSING
+
+- [paddleOCRCorrectOutputs](https://github.com/yuranusduke/paddleOCRCorrectOutputs)：Get the key-value of OCR recognition result (@ [yuranusduke](https://github.com/yuranusduke))
+
+### 1.2 NEW FEATURES FOR PaddleOCR
+
+- Thanks [authorfu](https://github.com/authorfu) for contributing Android([#340](https://github.com/PaddlePaddle/PaddleOCR/pull/340)) and [xiadeye](https://github.com/xiadeye) for contributing IOS demo code([#325](https://github.com/PaddlePaddle/PaddleOCR/pull/325)).
+- Thanks [tangmq](https://gitee.com/tangmq) for adding docker deployment service to PaddleOCR to support quick release of callable restful API services([#507](https://github.com/PaddlePaddle/PaddleOCR/pull/507)).
+- Thanks [lijinhan](https://github.com/lijinhan) for adding Java springboot to PaddleOCR and call OCR hubserving interface to complete the use of OCR service deployment([#1027](https://github.com/PaddlePaddle/PaddleOCR/pull/1027)).
+- Thanks [Evezerest](https://github.com/Evezerest), [ninetailskim](https://github.com/ninetailskim), [edencfc](https://github.com/edencfc), [BeyondYourself](https://github.com/BeyondYourself), [1084667371](https://github.com/1084667371)  for contributing  complete code of [PPOCRLabel](https://github.com/PaddlePaddle/PaddleOCR/blob/release/2.3/PPOCRLabel/README_ch.md).
+
+### 1.3 CODE AND DOCUMENT OPTIMIZATION
+
+- Thanks [zhangxin](https://github.com/ZhangXinNan)([Blog](https://blog.csdn.net/sdlypyzq)) for contributing new visualization methods and adding .gitgnore, handling the problem of manually setting the PYTHONPATH environment variable([#210](https://github.com/PaddlePaddle/PaddleOCR/pull/210)).
+- Thanks [lyl120117](https://github.com/lyl120117) for contributing code to print network structure([#304](https://github.com/PaddlePaddle/PaddleOCR/pull/304)).
+- Thanks [BeyondYourself](https://github.com/BeyondYourself) for making a lot of great suggestions for PaddleOCR and simplifying some code styles of paddleocr([so many commits)](https://github.com/PaddlePaddle/PaddleOCR/commits?author=BeyondYourself).
+- Thanks [Khanh Tran](https://github.com/xxxpsyduck) and [Karl Horky](https://github.com/karlhorky) for contributing modifing English documents.
+
+### 1.4 MULTILINGUAL CORPUS
+
+- Thanks [xiangyubo](https://github.com/xiangyubo)  for contributing handwritting Chinese OCR dataset([#321](https://github.com/PaddlePaddle/PaddleOCR/pull/321)).
+- Thanks [Mejans](https://github.com/Mejans) for contributing dictionary and corpus of the new language Occitan to PaddleOCR([#954](https://github.com/PaddlePaddle/PaddleOCR/pull/954)).
+
+## 2. CONTRIBUTION ILLUSTRATING
+
+### 2.1 NEW FUNCTION CLASS
+
+PaddleOCR welcomes community contributions to various services, deployment examples and software applications with paddleOCR as the core. Certified community contributions will be added to the above community contribution table to increase exposure for the majority of developers, which is also the glory of PaddleOCR, including:
+
+- Project form: the project code certified by the official community shall have good specifications and structure, and shall be equipped with a detailed README.md, which describes how to use the project. Through add a line 'paddleocr' to the requirements.txt, which can be automatically included in the usedby of paddleocr.
+
+- Integration method: if it is an update to the existing PaddleOCR tool, it will be integrated into the main repo. If a new function is expanded for paddleocr, please contact the official personnel first to confirm whether the project is integrated into the master repo, *even if the new function is not integrated into the master repo, we will also increase the exposure of your personal project in the way of community contribution.* 
+
+
+### 2.2 CODE OPTIMIZATION
+
+If you encounter code bugs and unexpected functions when using PaddleOCR, you can contribute your modifications to PaddleOCR, including:
+
+- Python code specifications are available for reference [Appendix 1:Python code specifications](./code_and_doc.md/#Appendix1).
+
+- Before submitting the code, please confirm again and again that no new bugs will be introduced, and describe the optimization points in the PR. If the PR solves an issue, please connect to the issue in the PR. All PR shall comply with the requirements in Appendix [3.2.10 Some conventions for submitting code.](./code_and_doc.md/#Some conventions for submitting code)
+
+- Please refer to the below before submitting. If you are not familiar with the git submission process, you can also refer to Section 3.2 of  [Appendix 3: description of Pull Request](./code_and_doc.md/#Appendix3).If you are not familiar with the git submission process, you can also refer to Section 3.2 of Appendix 3.
+
+**Finally, please add the label Third Party in the title of PR and @ Everest in the description , PR with this label will be treated with high priority`[third-part]`.**
+
+### 2.3 DOCUMENT OPTIMIZATION
+
+If you encounter problems such as unclear document description, missing description and invalid link when using PaddleOCR, you can contribute your modifications to PaddleOCR. For document writing specifications, please refer to [Appendix 2: document specifications](./code_and_doc.md/#Appendix2). **Finally, please add the label Third Party in the title of PR and @ Everest in the description , PR with this label will be treated with high priority`[third-party].**
+
+## 3. MORE CONTRIBUTION OPPORTUNITIES
+
+We encourage developers to use PaddleOCR to realize their ideas. At the same time, we also list some valuable development directions after analysis, which are collected in the regular season of community projects as a whole.
+
+## 4. CONTACT US
+
+We very much welcome developers to contact us before they intend to contribute code, documents, corpus and other contents to PaddleOCR, which can greatly reduce the communication cost in the PR process. At the same time, if you find some ideas difficult to realize personally, we can also recruit like-minded developers for the project in the form of SIG. Projects funded through SIG channels will receive deep R &amp; D support and operational resources (such as official account publicity, live broadcast lessons, etc.).
+
+Our recommended contribution process is:
+
+- By adding the `[Third Party]` mark in the topic of GitHub issue, explain the problems encountered (and the ideas to solve) or the functions to be expanded, and wait for the reply of the person on duty. For example, ` [Third Party] contributes IOS examples to PaddleOCR`.
+- After communicating with us and confirming that the technical scheme or bugs and optimization points are correct, add functions or modify them accordingly, and the codes and documents shall comply with relevant specifications.
+- PR links to the above issue and waits for review.
+
+## 5. THANKS AND FOLLOW-UP
+
+  - After the code is combined, the information will be updated in the first section of this document. The default link is GitHub name and home page. If you need to change the home page, you can also contact us.
+  - New important function classes will be advertised in the user group and enjoy the honor of the open source community.
+  - **If you have a PaddleOCR based project that does not appear in the above list, follow `4. CONTACT US` .**