Skip to content

Conversation

@oierlauzi
Copy link
Collaborator

@oierlauzi oierlauzi commented Oct 6, 2025

Currently we are experiencing issues with CUDA 12.5 onwards. Thus, we are detecting this condition in CMake to prematurely throw an error. In the future we need to add this compatibility, as CUDA >=12.5 is becoming increasingly more popular

Currently we are experiencing issues with CUDA 12.5 onwards. Thus, we are detecting this condition in CMake to prematurely throw an error.
@oierlauzi oierlauzi self-assigned this Oct 6, 2025
@oierlauzi oierlauzi added the bug Something isn't working label Oct 6, 2025
@albertmena
Copy link
Collaborator

@albertmena
Copy link
Collaborator

Could you add some message like:

    if(CUDAToolkit_VERSION VERSION_LESS "10.2" OR CUDAToolkit_VERSION VERSION_GREATER_EQUAL "12.5")
        message(WARNING "Found an incompatible CUDA ${CUDAToolkit_VERSION}, please install CUDA version 10.2 to 12.4. https://i2pc.github.io/docs/Installation/Requirements/index.html#cuda-optional")
        return()
    endif()

@oierlauzi
Copy link
Collaborator Author

Could you add some message like:

    if(CUDAToolkit_VERSION VERSION_LESS "10.2" OR CUDAToolkit_VERSION VERSION_GREATER_EQUAL "12.5")
        message(WARNING "Found an incompatible CUDA ${CUDAToolkit_VERSION}, please install CUDA version 10.2 to 12.4. https://i2pc.github.io/docs/Installation/Requirements/index.html#cuda-optional")
        return()
    endif()

This would never be reached because find_package will not populate CUDAToolkit_VERSION if it does not find an appropriate version. However it will generate its own warning with a message similar to this one. Nevertheless this PR is still Work in Prorgress

@albertmena
Copy link
Collaborator

albertmena commented Oct 17, 2025

For you to know: https://xmipp.i2pc.es/api/xmipp/version-CUDA/

There are several Xmipp installations with CUDA > 12.4

A wonderful view of it: https://xmipp.i2pc.es/metrics

@sonarqubecloud
Copy link

@albertmena
Copy link
Collaborator

I understand, only version 12.5 fails, the higher and lower versions work.

@oierlauzi
Copy link
Collaborator Author

oierlauzi commented Oct 21, 2025

Maybe we can disregard this PR and wait to see why it fails sometimes with CUDA 12.5+

@albertmena albertmena closed this Oct 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants