From dc20fcb88ecdbb8970599b84efd74711a14b9c9b Mon Sep 17 00:00:00 2001 From: asideofcode-dev <133222359+asideofcode-dev@users.noreply.github.com> Date: Wed, 8 Jan 2025 22:05:06 +0000 Subject: [PATCH] Revert prefactor m in autograd documentation The prefactor m was incorrectly added to the vector-Jacobian product formula in the autograd tutorial. This change was based on the mistaken assumption that m-scaling was necessary to account for multiple terms in the summation. However, the chain rule and vector-Jacobian product formula already correctly aggregate contributions from all intermediate variables without requiring explicit scaling by m. --- beginner_source/blitz/autograd_tutorial.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/beginner_source/blitz/autograd_tutorial.py b/beginner_source/blitz/autograd_tutorial.py index b31a30409aa..b736b429eee 100644 --- a/beginner_source/blitz/autograd_tutorial.py +++ b/beginner_source/blitz/autograd_tutorial.py @@ -191,7 +191,7 @@ # .. math:: # # -# J^{T}\cdot \vec{v} = m \cdot \left(\begin{array}{ccc} +# J^{T}\cdot \vec{v} = \left(\begin{array}{ccc} # \frac{\partial y_{1}}{\partial x_{1}} & \cdots & \frac{\partial y_{m}}{\partial x_{1}}\\ # \vdots & \ddots & \vdots\\ # \frac{\partial y_{1}}{\partial x_{n}} & \cdots & \frac{\partial y_{m}}{\partial x_{n}} @@ -199,7 +199,7 @@ # \frac{\partial l}{\partial y_{1}}\\ # \vdots\\ # \frac{\partial l}{\partial y_{m}} -# \end{array}\right) = m \cdot \left(\begin{array}{c} +# \end{array}\right) = \left(\begin{array}{c} # \frac{\partial l}{\partial x_{1}}\\ # \vdots\\ # \frac{\partial l}{\partial x_{n}}