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

Add cuda-12 host-configs for Lassen #3067

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from

Conversation

victorapm
Copy link
Contributor

Copy link

codecov bot commented Apr 5, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 53.19%. Comparing base (ddf3e97) to head (537bb8c).

Files Patch % Lines
...es/hypre/mgrStrategies/MultiphasePoromechanics.hpp 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #3067   +/-   ##
========================================
  Coverage    53.19%   53.19%           
========================================
  Files          989      989           
  Lines        83647    83646    -1     
========================================
  Hits         44498    44498           
+ Misses       39149    39148    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@TotoGaz
Copy link
Contributor

TotoGaz commented Apr 5, 2024

@sframba Do you know if CUDA 12 is supported on Pangea3
@matteofrigo5 Same question on Sherlock

set(ENABLE_CUDA_NVTOOLSEXT ON CACHE BOOL "")

# Overwrite ESSL defaults from lassen-base.cmake
# Reason: libesslsmpcuda.so depends on cuda-11
Copy link
Collaborator

Choose a reason for hiding this comment

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

so we don't really need it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

libesslsmpcuda.so? Yes, geos doesn't complain if I remove it. It seems it's not being used anywhere

@sframba
Copy link
Contributor

sframba commented Apr 9, 2024

@sframba Do you know if CUDA 12 is supported on Pangea3 @matteofrigo5 Same question on Sherlock

Not sure, we'll have to ask the IBM support. We have upgraded to cuda 11.5.0 at the end of 2023. Is cuda 12 needed to solve the externalSolvers unit test issue? Or to access some new hypre features on GPU?

@matteofrigo5
Copy link
Contributor

I don't see any constraints regarding Sherlock. We have these four versions of CUDA 12 installed: (12.0.0 12.1.1 12.2.0 12.4.0). @victorapm, do you suggest any one in particular?

@TotoGaz
Copy link
Contributor

TotoGaz commented Apr 10, 2024

Not sure, we'll have to ask the IBM support. We have upgraded to cuda 11.5.0 at the end of 2023. Is cuda 12 needed to solve the externalSolvers unit test issue? Or to access some new hypre features on GPU?

It's mainly for this.

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.

None yet

5 participants