Skip to content

Add support for mfem cuda build#35608

Merged
alalazo merged 2 commits intospack:developfrom
aaroncblack:feature/aaroncblack/mfem_4.5_cuda_support
Mar 8, 2023
Merged

Add support for mfem cuda build#35608
alalazo merged 2 commits intospack:developfrom
aaroncblack:feature/aaroncblack/mfem_4.5_cuda_support

Conversation

@aaroncblack
Copy link
Copy Markdown
Contributor

Add missing 'cublas' dependency in MFEM library spack build.

@aaroncblack
Copy link
Copy Markdown
Contributor Author

aaroncblack commented Feb 22, 2023

@v-dobrev @tzanio I spoke briefly with Veselin about this in the past. Cublas is a dependency on newer hypre versions for the cuda build.

Veselin, I wasn't sure if you had a planned change already for this spack package but I went ahead and created this PR.

v-dobrev
v-dobrev previously approved these changes Feb 23, 2023
Copy link
Copy Markdown
Member

@v-dobrev v-dobrev left a comment

Choose a reason for hiding this comment

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

Thanks @aaroncblack, this looks good and it should be fine with older hypre versions too.

@aaroncblack
Copy link
Copy Markdown
Contributor Author

aaroncblack commented Feb 23, 2023

@v-dobrev I'm not sure who has permissions to merge this. (I don't). I'm guessing you and @tzanio do, as the mfem owners, but if 'Merge pull request' is greyed out with "You’re not authorized to merge this pull request." let me know. I can ask Todd Gamblin about it.

v-dobrev
v-dobrev previously approved these changes Feb 23, 2023
@v-dobrev
Copy link
Copy Markdown
Member

I think I have permission to merge this after all PR checks pass, however I typically wait for someone from the Spack team to do it since I'm not familiar with all the policies on merging.

Copy link
Copy Markdown
Member

@alalazo alalazo left a comment

Choose a reason for hiding this comment

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

@v-dobrev already approved

@alalazo alalazo enabled auto-merge (squash) March 1, 2023 13:18
@v-dobrev
Copy link
Copy Markdown
Member

v-dobrev commented Mar 7, 2023

@alalazo, I tried hitting the "Update branch" button to get clean CI checks to trigger the auto-merge but that did not work. What can we do to get this merged?

@alalazo
Copy link
Copy Markdown
Member

alalazo commented Mar 8, 2023

@spackbot run pipeline

@spackbot-app
Copy link
Copy Markdown

spackbot-app bot commented Mar 8, 2023

I've started that pipeline for you!

@alalazo alalazo self-assigned this Mar 8, 2023
@alalazo
Copy link
Copy Markdown
Member

alalazo commented Mar 8, 2023

Tried re-running the two spuriously failed builds.

@alalazo alalazo merged commit 81765e0 into spack:develop Mar 8, 2023
jmcarcell pushed a commit to key4hep/spack that referenced this pull request Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants