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 hipBLAS feature and fix build script #337

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

AdamNiederer
Copy link

@AdamNiederer AdamNiederer commented Jun 10, 2024

Attempts to copy the CMake build steps as much as possible. GPU is detected and models are loaded into VRAM but it appears to segfault when doing anything else. I'm at a bit of a loss on the segfault but this is definitely better than before.

@MarcusDunn
Copy link
Contributor

Attempts to copy the CMake build steps as much as possible. GPU is detected and models are loaded into VRAM but it appears to segfault when doing anything else. I'm at a bit of a loss on the segfault but this is definitely better than before.

So to be clear, if I merge this, it still segfaults with HIP, just later?

@AdamNiederer
Copy link
Author

So to be clear, if I merge this, it still segfaults with HIP, just later?

Correct, previously it would not build at all; now it builds, links, and can run some things, but will quickly segfault on the GPU when attempting to do anything of consequence. I've been unable to pinpoint why so hopefully somebody more knowledgeable can use this as a head start.

@MarcusDunn
Copy link
Contributor

Do you befit from having this merged, or is it fine here as a starting point for someone who wants to bring it across the finish line?

I'd prefer to not give the impression of HIP support if it's known to not work, but if it's useful then I don't mind merging.

@AdamNiederer
Copy link
Author

I'm fine with leaving keeping a local dependency for now since I'm not able to get my workload running. I'll mark this as a draft

@AdamNiederer AdamNiederer marked this pull request as draft June 15, 2024 02:33
Attempts to copy the CMake build steps as much as possible. GPU is detected when
running but on my setup appears to segfault when loading the model. I'm at a bit
of a loss on the segfault but this is definitely better than before.
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