Layout-Parser / layout-parser

A Unified Toolkit for Deep Learning Based Document Image Analysis
https://layout-parser.github.io/
Apache License 2.0
4.64k stars 449 forks source link

Improper Documentation #195

Closed Mukilan-Krishnakumar closed 5 months ago

Mukilan-Krishnakumar commented 10 months ago

Describe the bug Under the Deep Layout Parsing section of the docs, the Detectron2LayoutModel is directly imported from Layout Parser but in actuality, it is imported from the models subclass of Layout Parser

Checklist

  1. I have searched related issues but cannot get the expected help.
  2. The bug has not been fixed in the latest version, see the Layout Parser Releases

To Reproduce Steps to reproduce the behavior: Visit this link and see it for yourself.

Environment

  1. Mac OS
  2. v0.3.4
  3. You may add addition that may be helpful for locating the problem, such as
    • How you installed PyTorch [e.g., pip, conda, source]
    • Other environment variables that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Error traceback If applicable, paste the error traceback here.

Screenshots

Screenshot 2023-09-01 at 10 26 49 AM

Additional context Add any other context about the problem here.

Mukilan-Krishnakumar commented 10 months ago

All we need to do is change the index.rst file, which can be found here

drewm23 commented 9 months ago

Agreed. Spent a bunch of time trying to debug why my code wasn't working.

nkrot commented 5 months ago

This library seems not be maintained any more. Looks like the interest in maintaining it dissipated once the paper was published.

A quotation for their paper (section 6 Conslusion):

< The LayoutParser team is committed to keeping the library updated continuously and bringing the most recent advances in DL-based DIA, such as multi-modal document modeling (an upcoming priority), to diverse audience of of end-users.

Publish or perish?

Mukilan-Krishnakumar commented 5 months ago

Seems like they won't be responding, so I am going to close this.