Skip to content

Conversation

@wsmoses
Copy link
Member

@wsmoses wsmoses commented Oct 19, 2025

No description provided.

end

if cuname == ""
return nothing
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
return nothing
return nothing

Comment on lines +126 to +129
if build_backend == "cuda"
throw(AssertionError("Could not detect cuda version, but requested cuda with auto version build"))
end
build_backend = "cpu"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
if build_backend == "cuda"
throw(AssertionError("Could not detect cuda version, but requested cuda with auto version build"))
end
build_backend = "cpu"
if build_backend == "cuda"
throw(
AssertionError(
"Could not detect cuda version, but requested cuda with auto version build",
),
)
end
build_backend = "cpu"

Comment on lines +131 to +135
if Int(get_cuda_version().major) == 13
build_backend = "cuda13"
else
build_backend = "cuda12"
end
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
if Int(get_cuda_version().major) == 13
build_backend = "cuda13"
else
build_backend = "cuda12"
end
if Int(get_cuda_version().major) == 13
build_backend = "cuda13"
else
build_backend = "cuda12"
end

@wsmoses wsmoses merged commit 4f2deaf into main Oct 19, 2025
10 of 12 checks passed
@wsmoses wsmoses deleted the ljl branch October 19, 2025 07:56
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