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

[SYCL] Update rpc-server.cpp to include SYCL backend #7682

Merged
merged 2 commits into from
Jun 2, 2024

Conversation

nickp27
Copy link
Contributor

@nickp27 nickp27 commented Jun 1, 2024

PR to address inclusion of SYCL backend for RPC server

Draft PR to address inclusion of SYCL backend for RPC server
@nickp27 nickp27 changed the title Update rpc-server.cpp to include SYCL backend [SYCL] Update rpc-server.cpp to include SYCL backend Jun 2, 2024
@@ -81,6 +85,14 @@ static ggml_backend_t create_backend() {
}
#endif

#elif GGML_USE_SYCL
Copy link
Collaborator

Choose a reason for hiding this comment

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

this will give a compile error if you don't remove the #endif on line 86

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should be resolved now!

@rgerganov rgerganov merged commit 9422c5e into ggerganov:master Jun 2, 2024
67 checks passed
@slaren
Copy link
Collaborator

slaren commented Jun 6, 2024

Does this really work? The SYCL backend has extras that are initialized during init_tensor.

@nickp27
Copy link
Contributor Author

nickp27 commented Jun 6, 2024

It's currently failing - see #7718, although I suspect that issue is only one of a number possibly, having spent a few hours trying to get it working and seeing that similar issues have persisted with whisper.cpp for the last couple of months

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants