-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
feat: integrate got-ocr2.0 as image reader #355
base: main
Are you sure you want to change the base?
Conversation
@phv2312, can you add a docker-compose file (allow choose the docker image for OCR service)? I think it will help people test more easily. |
25904f0
to
de703a3
Compare
Hi @taprosoft @cin-niko. Sorry for no update for long time. Can you help to review this PR again |
a488e31
to
a358fd2
Compare
Hi @cin-niko and @taprosoft . I have updated according to niko's comments and rebased from the latest master already. |
@phv2312 Overall is good. But it seems that setting the loader for extensions feature doesn't work.
|
@phv2312 sorry for the late comment. Overall the logic is fine but the current settings UI is a bit cluttered. I will push a small change to improve this prior to merging. |
Description
Type of change
Checklist