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

feature request: Mac OS and Linux ARM64 compatibility? #95

Open
pettyalex opened this issue Mar 26, 2024 · 2 comments
Open

feature request: Mac OS and Linux ARM64 compatibility? #95

pettyalex opened this issue Mar 26, 2024 · 2 comments

Comments

@pettyalex
Copy link

I've had multiple researchers on my team lose some time this week assuming that shapeit5 is Mac OS compatible, but it is not. At the same time, the major cloud vendors continue to offer ARM64 processors at very significant discounts compared to Intel or AMD, meaning that even if vector performance is worse they can offer a better value.

At the same time, bioconda is starting to widely roll out ARM64 support not just for Mac OS, but also Linux: bioconda/bioconda-recipes#23454 .

I'd be glad to update a PR that I developed and validated that enables ARM64 compatibility for Shapeit5 if you are interested in merging it: #68 . I think there might be a surprising amount of users out there that would love to have this.

@pettyalex
Copy link
Author

I have one more thought: it's very likely that in the future, if shapeit does not merge ARM64 compatibility upstream in this repo, different outside packaging groups will start patching it in. I've seen that happen already for other genomics tools, and it would save a whole lot of extra work if we could get it into the upstream project instead.

@rwk-unil
Copy link
Contributor

Hello,
thank you for suggesting this. SHAPEIT5 is Mac OS compatible, at least for Intel based Macs. As I only have an Intel based Mac and have no access to an ARM64 Mac I was not able to try it out.

For the cloud we target Intel as it was the most common architecture, Mac OS in the cloud is rather expensive compared to Linux based instances so we did not bother. But your suggestion about ARM64 cloud instances makes a lot of sense, as indeed we see a shift towards very efficient ARM64 machines (e.g., AWS Graviton).

If you have the changes for ARM64 locally already, I think it is a good idea to issue a PR or update your current PR. I have several Linux based ARM64 machines so I can try it out and I'll update @srubinacci for merging this into the mainline repo.

Regards,
Rick

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

No branches or pull requests

2 participants