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

Upgrade to Ubuntu 24.04 and Cuda 12.8.0 #1488

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

badisa
Copy link
Collaborator

@badisa badisa commented Feb 7, 2025

  • Would have to upgrade to at least Cuda 12.5.1 to get Ubuntu 24.04 images, unless we want to build our own. With the release of the Nvidia RTX 50 series, upgrading to 12.8.0 seemed to make more sense
  • A warning in the latest NVCC complains about how we handle our templating code. We have been inconsistent about this, but the practice seems to be to define the implementation of templated function in .cuh then declare the usage in .cu.

Example NVCC Error

/code/timemachine/timemachine/cpp/src/kernels/k_sampling.cuh(27): error #20281-D: in whole program compilation mode ("-rdc=false"), a global function template instantiation or specialization ("timemachine::k_setup_gumbel_max_trick_targeted_insertion ") will be required to have a definition in the current translation unit, when "-static-global-template-stub" will be set to "true" by default in the future. To resolve this issue, either use "-rdc=true", or explicitly set "-static-global-template-stub=false" (but see nvcc documentation about downsides of turning it off)

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.

1 participant