ggerganov / ggml

Tensor library for machine learning
MIT License
11.25k stars 1.05k forks source link

Update simple-backend.cpp #1011

Open paul-tian opened 1 week ago

paul-tian commented 1 week ago

Fix the printing logic of the result matrix.

paul-tian commented 1 week ago

The original code is not reflecting the correct result of matrices multiplication. I've updated the code to reflect the correct result, and also updated the corresponding comments in the code.

BorisChuev commented 1 week ago

Sent with OfficeSuite Mail for Windows PC ------------- Original Message --------------From: Yuzhe Tian @.>Sent: 10.11.2024 10:59:32To: ggerganov/ggml @.>Cc: Subscribed @.***>Subject: Re: [ggerganov/ggml] Update simple-backend.cpp (PR #1011)

The code in this file is not reflecting the correct result of matrices multiplication. I've updated the code to reflect the correct result, and also updated the corresponding comments in the code.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: @.***>

--=-Yi+HcrUTfDeSQytnJwL2mQ== Content-Type: text/html; charset=utf-8 Content-Id: @.***> Content-Transfer-Encoding: quoted-printable



Sent with OfficeSuite Mail for Windows PC

------------- Original Messag= e --------------
From: Yuzhe Tian ***@***.***>
Sent: 10.11.2024 10:59:32
To: ggerganov/ggml <= ***@***.***>
Cc: Subscribed ***@***.***><= /div>
Subject: Re: [gg= erganov/ggml] Update simple-backend.cpp (PR #1011)
=


The code in this file = is not reflecting the correct result of matrices multiplication. I've updat= ed the code to reflect the correct result, and also updated the correspondi= ng comments in the code.

=E2=80=94
Reply to this email directly, view = it on GitHub, or unsubscribe.
You are receivi= ng this because you are subscribed to this thread.3D""Message ID:= <ggerganov/ggml/pull/1011/c2466629564@g= ithub.com>

= --=-Yi+HcrUTfDeSQytnJwL2mQ==--