Skip to content
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

OCR Block v2 #706

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

OCR Block v2 #706

wants to merge 8 commits into from

Conversation

stellasphere
Copy link
Contributor

Description

Multi-model support of the OCR block

Draft Only

List any dependencies that are required for this change.

Type of change

  • New feature (non-breaking change which adds functionality)

How has this change been tested, please provide a testcase or example of how you tested the change?

YOUR_ANSWER

Any specific deployment considerations

For example, documentation changes, usability, usage/costs, secrets, etc.

Docs

  • Docs updated? What were the changes:

@PawelPeczek-Roboflow
Copy link
Collaborator

Collision with #692
I am not sure about the status of work for the person asked about contribution

RUN pip install watchdog[watchmedo]
ENTRYPOINT watchmedo auto-restart --directory=/app/inference --pattern=*.py --recursive -- uvicorn cpu_http:app --workers $NUM_WORKERS --host $HOST --port $PORT
Copy link
Collaborator

Choose a reason for hiding this comment

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

from inference.core.workflows.prototypes.block import BlockResult


class BaseOCRModel(ABC):
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please do not share common base class for OCR blocks

@PawelPeczek-Roboflow
Copy link
Collaborator

@stellasphere lets push that through finish line this week, ok?

@PawelPeczek-Roboflow
Copy link
Collaborator

@stellasphere ping

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.

2 participants