[discussion] Plugins docTR #1024
Replies: 2 comments 2 replies
-
Passing plugins to the predictor is a very good idea 👍 We certainly need to simplify the codebase structure for upcoming new features though. As long as those plugins don't require extra dependencies, I think it can be kept inside docTR. And that raises a question of mine:
What do you think? |
Beta Was this translation helpful? Give feedback.
-
@felixdittrich92 I share @frgfm's point of view: I think it should remain in the same repo. The problem of plugins is that you need to update each repo to keep them up-to-date. It would be easier to update thanks to CI. Wouldn't it be possible to let from doctr.object_detector import ObjectDetector
from doctr.language_detector import LanguageDetector
from doctr.table_recognizer import TableRecognizer and to use We can also build some around "detector theme", as it has been done in VidGear with "Gears". |
Beta Was this translation helpful? Give feedback.
-
Hi 👋,
with the growing number of features which are not directly related to text detection and recognition (like object detection / language detection / and following) the repository is also growing to provide an easier way to maintain we should maybe think about an kind of plugins which provide the functionality the users want:
To explain this a bit more in detail the idea i have in mind is something like:
We could then adjust the output according to the plugins
Some improvements i would see:
This is just a rough idea, so what do you think? @frgfm @charlesmindee @odulcy-mindee @aminemindee
Beta Was this translation helpful? Give feedback.
All reactions