Skip to content

Delegate Tesseract build to ocrd_tesserocr#409

Merged
kba merged 10 commits into
OCR-D:masterfrom
bertsky:delegate-tesseract-build
Mar 7, 2024
Merged

Delegate Tesseract build to ocrd_tesserocr#409
kba merged 10 commits into
OCR-D:masterfrom
bertsky:delegate-tesseract-build

Conversation

@bertsky

@bertsky bertsky commented Feb 22, 2024

Copy link
Copy Markdown
Collaborator

Robert Sachunsky added 6 commits February 22, 2024 22:52
- remove rules and defs for Tesseract build
- delegate targets for ocrd_tesserocr executables (including Tesseract CLI)
  to ocrd_tesserocr rules
- keep targets `{clean,install}-tesseract` and `install-tesseract-training`
  (also delegating to ocrd_tesserocr rules), but decouple from `all`
- remove tesseract and tesserocr submodules
- ensure ocrd_tesserocr is checkout out recursively, too
@bertsky

bertsky commented Feb 22, 2024

Copy link
Copy Markdown
Collaborator Author

@kba do you have any idea why CI does not trigger on this PR? In CircleCI config, forked PR building is still enabled, but I cannot even see the PR...

@bertsky bertsky force-pushed the delegate-tesseract-build branch from 670a6cf to 7f19746 Compare February 23, 2024 00:20
@bertsky bertsky force-pushed the delegate-tesseract-build branch from 7f19746 to 144b642 Compare February 23, 2024 00:20
@bertsky bertsky requested review from joschrew and kba March 5, 2024 21:06
@bertsky

bertsky commented Mar 6, 2024

Copy link
Copy Markdown
Collaborator Author

I don't know why it does not trigger the CI here, but at least it did on my fork.

@joschrew joschrew left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for notifying me. I have looked through the code and I am currently trying to test by building the Dockerimage. Probably it is not necessary to wait for me here.

@kba kba merged commit b9d6e31 into OCR-D:master Mar 7, 2024
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.

model download in Docker only allowed for root /models not working Installation of models only works when tesseract is in module list

3 participants