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

Docker: build multi-architecture images #432

Open
bertsky opened this issue May 16, 2024 · 3 comments
Open

Docker: build multi-architecture images #432

bertsky opened this issue May 16, 2024 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@bertsky
Copy link
Collaborator

bertsky commented May 16, 2024

ATM we only target linux/amd64 as platform. This unnecessarily reduces the utility of OCR-D. For example, Apple hardware will almost always be some ARM CPU (becoming slow under QEMU or not even supported because of missing AVX). Also, in HPC systems the architecture can be PowerPC or other non-x86 – converting Docker images to Singularity is not possible if the CPU architecture does not match.

I suggest following the second recipe (parallel+merge) from this guide to address this.

@stweil
Copy link
Collaborator

stweil commented May 16, 2024

I run OCR-D natively on macOS with ARM64 and would support multi-architecture Docker images only if there is a certain demand for such images. Otherwise I am afraid that this simply wastes resources (developer time, environmental resources needed for increased build time, image sizes, ...).

@bertsky
Copy link
Collaborator Author

bertsky commented May 16, 2024

Native installation is more demanding for users. The whole point of having Dockerization is reducing the threshold for the user. We already decided to go for that when we set up the Docker route. It's just a matter of following through on the CPU architecture.

And on resources: Build time – certainly, but it's worth it (see above). Developer time – sure, but not so much (it's only a minor change), and supporting users with their native installation struggle also binds time. We still have lots of issues with the Docker builds ATM, so IMO this can all happen in one move.

@stweil stweil added the enhancement New feature or request label May 23, 2024
@bertsky
Copy link
Collaborator Author

bertsky commented Jul 31, 2024

So now that OCR-D/core#1239 is merged, we can proceed doing the same here.

@bertsky bertsky self-assigned this Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants