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

Setting tensor's backend #779

Closed
rgerganov opened this issue Apr 1, 2024 · 2 comments
Closed

Setting tensor's backend #779

rgerganov opened this issue Apr 1, 2024 · 2 comments

Comments

@rgerganov
Copy link
Collaborator

After recent CUDA refactoring, we no longer set GGML_BACKEND_TYPE_GPU in ggml_backend_cuda_buffer_init_tensor() and tensors end up with GGML_BACKEND_TYPE_CPU backend even when using CUDA.

Is this a bug or part of a bigger plan which removes the backend property of the tensors?

@slaren
Copy link
Collaborator

slaren commented Apr 1, 2024

The goal is to remove backend from the tensor entirely. Backends can use the buffer type to identify the storage location of the tensors instead. I think that at this point, only the OpenCL backend depends on the value of the backend property.

@rgerganov
Copy link
Collaborator Author

Thanks. That confirms my assumption.

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

No branches or pull requests

2 participants