-
Notifications
You must be signed in to change notification settings - Fork 522
New version: CUDA v2.5.0 (registered via LocalRegistry.jl) #27666
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
Conversation
c00bf21 to
87c82ca
Compare
|
@maleadt This is ready to review. |
87c82ca to
3613668
Compare
UUID: 052768ef-5323-5732-b1bb-66c8b64840ba Tree: 2ed30f30c8905cea954b0904a98211745b3818f5
3613668 to
cd62ca9
Compare
maleadt
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks!
This reverts commit 589c280.
|
I think this has caused bad things to happen in packages that depend on CUDA.jl when you try to run those packages on Julia < 1.6. The reason is that the UUID Can we register a LazyArtifacts.jl package with this UUID in the General registry? So that packages that depend on CUDA.jl don't break on Julia < 1.6? cc: @staticfloat @giordano @maleadt See e.g. https://github.com/JuliaGPU/KernelAbstractions.jl/runs/1680372624 for a broken build on Julia 1.4:
|
|
We could probably even have a LazyArtifacts "package" that has zero registered versions. So it can't actually be installed, but at least the resolver on Julia < 1.6 can map the UUID |
|
Register please? And since the name LazyArtifacts is a stdlib in 1.6, I think it's fine to merge the registration of https://github.com/JuliaPackaging/LazyArtifacts.jl immediately. |
|
Oh boy merging this PR broke the entire Julia ecosystem 😬 At least it got reverted pretty quickly. |
I don't have the time to do it right now, so please go ahead if you can do it. It'd also be good to test it actually works across all julia versions (well, v1.3+ I guess?), I never got around that |
|
The fact is that the package is likely completely useless in v1.3-v1.5. I added the same exports as in the standard library (at least those actually available in v1.5), for consistency, but perhaps an empty package is good enough. Someone needs to decide what to do in practice |
|
For the use case here (being able to register CUDA 2.5.0 in the General registry without breaking the General registry for everyone on Julia < 2.5), I think it's sufficient to register LazyArtifacts.jl without any versions. This is done here: #27753 It would be good to get a sanity check to confirm that it's safe to register a package without any versions. |
This pull request uses LocalRegistry.jl to register CUDA.jl version 2.5.0 from the
masterbranch of CUDA.jl.For posterity, here is the Bash script that I ran to generate this pull request. I used the latest Julia master.
And here is the output: (click to expand)
After running the Bash script, I went to the GitHub website and created a new pull request to merge the
dpa/register-cuda-2.5.0branch intomaster.