Skip to content

CANN: Add ggml_set_rows #14943

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

Merged
merged 1 commit into from
Jul 29, 2025
Merged

CANN: Add ggml_set_rows #14943

merged 1 commit into from
Jul 29, 2025

Conversation

hipudding
Copy link
Collaborator

Add ggml_set_rows for F32 and F16. #14274 #14661

Make sure to read the contributing guidelines before submitting a PR

@github-actions github-actions bot added ggml changes relating to the ggml tensor library for machine learning Ascend NPU issues specific to Ascend NPUs labels Jul 29, 2025
@hipudding hipudding requested a review from ggerganov July 29, 2025 11:55
@hipudding hipudding assigned hipudding and unassigned hipudding Jul 29, 2025
Add ggml_set_rows for F32 and F16. ggml-org#14274 ggml-org#14661
@hipudding
Copy link
Collaborator Author

Due to lack of ACL operator support, currently only float and float16 data types are supported. Support for other data types will be added once the operators are available. Performance verification on Qwen2.5-7B FP16 shows a 3% improvement.

@noemotiovon
Copy link
Contributor

noemotiovon commented Jul 29, 2025

Great contribution! Thank you for the code refactoring and the addition of the SET_ROWS operator.

@hipudding hipudding merged commit 204f2cf into ggml-org:master Jul 29, 2025
48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ascend NPU issues specific to Ascend NPUs ggml changes relating to the ggml tensor library for machine learning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants