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

Updated for Torch v1.10.2, and NNlib >= v0.8 #64

Merged
merged 4 commits into from
Nov 16, 2024

Conversation

stemann
Copy link
Collaborator

@stemann stemann commented Nov 6, 2024

Replaced Torch_jll v1.4 with TorchCAPI_jll v0.2; TorchCAPI_jll v0.2 is supported by Torch_jll v1.10.2.

Torch_jll uses extended platform selection (platform augmentation) which requires Julia 1.7+.

Updated Julia wrapper using Julia wrapper generator (using Julia 1.11).

Updated usages of functions changed in #61 (Updated C wrapper): E.g., at_grad_set_enabled.

Also:

  • Changed tests to make CUDA optional
  • CI: Buildkite: Added testing on CUDA 11.3, and Julia 1.10, and 1.11
  • CI: Added GitHub Actions workflow for running tests

Resolves #17
Resolves #42
Resolves #50
Resolves #54

Also,

@stemann stemann mentioned this pull request Nov 6, 2024
@stemann stemann force-pushed the feature/torch_1.10.2 branch 5 times, most recently from 3db9d60 to 5d1cfc1 Compare November 7, 2024 00:07
@stemann stemann force-pushed the feature/torch_1.10.2 branch 17 times, most recently from b8c7c7b to 4afecd8 Compare November 14, 2024 23:17
Copy link

codecov bot commented Nov 14, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@stemann stemann force-pushed the feature/torch_1.10.2 branch from 4afecd8 to 78fcb8f Compare November 15, 2024 08:22
@stemann stemann changed the title WIP Updated for Torch v1.10.2 Updated for Torch v1.10.2, and NNlib >= v0.8 Nov 15, 2024
@stemann stemann marked this pull request as ready for review November 15, 2024 08:24
@stemann stemann force-pushed the feature/torch_1.10.2 branch from 78fcb8f to f547c13 Compare November 15, 2024 08:46
@stemann
Copy link
Collaborator Author

stemann commented Nov 15, 2024

Will merge this tomorrow if there are no objections.

@stemann
Copy link
Collaborator Author

stemann commented Nov 15, 2024

Will to some extent also remedy #20 - a re-build of Torch_jll using lazy artifacts is required before CUDA can be optional on Linux x86_64: JuliaPackaging/Yggdrasil#9785

@stemann stemann force-pushed the feature/torch_1.10.2 branch from f547c13 to b3eb223 Compare November 16, 2024 21:40
Replaced Torch_jll v1.4 with TorchCAPI_jll v0.2; TorchCAPI_jll v0.2 is supported by Torch_jll v1.10.2.

Torch_jll uses extended platform selection (platform augmentation) which requires Julia 1.7+.

Updated Julia wrapper using Julia wrapper generator (using Julia 1.11).

Updated usages of functions changed in FluxML#61 (Updated C wrapper): at_grad_set_enabled.
Testing on Julia 1.6 with CUDA is no longer possible, as the extended platform selection (platform augmentation) requires Julia 1.7+.
@stemann stemann force-pushed the feature/torch_1.10.2 branch from b3eb223 to 77ecce7 Compare November 16, 2024 21:43
@stemann stemann merged commit 54c3ca3 into FluxML:master Nov 16, 2024
8 of 9 checks passed
@stemann stemann deleted the feature/torch_1.10.2 branch November 16, 2024 22:02
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.

Update wrapper Failed installing Torch.jl Error upon installing Required version of glibc?
1 participant