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

READY FOR ARM RUNNERS #22

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

johnnynunez
Copy link

No description provided.

@jacquev6
Copy link
Member

Hello @johnnynunez, thank you for wanting to contribute to Lincs!

Can you please give me a little more context about the change you're submitting? What problem are you solving? How does this change solve your problem? What level of testing have you done?

Thanks!

@johnnynunez
Copy link
Author

Hello @johnnynunez, thank you for wanting to contribute to Lincs!

Can you please give me a little more context about the change you're submitting? What problem are you solving? How does this change solve your problem? What level of testing have you done?

Thanks!

Hello,
Now you can do support for gh200 and nvidia arm devices.
Arm laptops, jetson, gh200 etc

@jacquev6
Copy link
Member

I see, thanks. I'll take a deeper look at the consequences of this change next time I spend time on the project.

@jacquev6
Copy link
Member

jacquev6 commented Jan 27, 2025

Note to myself, a few other things that need to happen before lincs can run on ARM on Linux (not exhaustive, just what comes to mind right now):

  • build wheels for ARM on Linux (can be done in Docker+Qemu, which is very long, or on ARM runners for GitHub Actions, not yet available AFAIK)
  • replicate the CUDA upgrade to the other GitHub Actions workflows (as hinted by this comment)
  • fix setup.py to use the new version of CUDA (not sure if I remember correctly but I think there is a hard-coded CUDA version there)
  • test extensively that the CUDA upgrade does not change lincs' numerical behavior in unwanted proportions

@johnnynunez
Copy link
Author

Note to myself, a few other things that need to happen before lincs can run on ARM on Linux (not exhaustive, just what comes to mind right now):

  • build wheels for ARM on Linux (can be done in Docker+Qemu, which is very long, or on ARM runners for GitHub Actions, not yet available AFAIK)
  • replicate the CUDA upgrade to the other GitHub Actions workflows (as hinted by this comment)
  • fix setup.py to use the new version of CUDA (not sure if I remember correctly but I think there is a hard-coded CUDA version there)
  • test extensively that the CUDA upgrade does not change lincs' numerical behavior in unwanted proportions

Note to myself, a few other things that need to happen before lincs can run on ARM on Linux (not exhaustive, just what comes to mind right now):

  • build wheels for ARM on Linux (can be done in Docker+Qemu, which is very long, or on ARM runners for GitHub Actions, not yet available AFAIK)
  • replicate the CUDA upgrade to the other GitHub Actions workflows (as hinted by this comment)
  • fix setup.py to use the new version of CUDA (not sure if I remember correctly but I think there is a hard-coded CUDA version there)
  • test extensively that the CUDA upgrade does not change lincs' numerical behavior in unwanted proportions

https://github.blog/changelog/2025-01-16-linux-arm64-hosted-runners-now-available-for-free-in-public-repositories-public-preview/

github/roadmap#1098

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