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

pg. 84 #65

Closed
alextzik opened this issue Apr 30, 2022 · 3 comments
Closed

pg. 84 #65

alextzik opened this issue Apr 30, 2022 · 3 comments
Assignees

Comments

@alextzik
Copy link

  • Exercise 5.2 mentions using a unit step factor, but the magnitude of the step is not 1 in the solution.

  • The solution of Exercise 5.7 uses the normalized direction of steepest descent, but the textbook derives the conjugate gradient using the unnormalized gradient direction.

@tawheeler
Copy link
Contributor

The first bullet has already been fixed in the errata

@tawheeler tawheeler self-assigned this Apr 30, 2022
@tawheeler
Copy link
Contributor

@mykelk I see three options here:

  1. do nothing - the questions specifies normalized. This is inconsistent though, so I'd rather do one of the other two.
  2. Reinforce that we are doing this with the normalized gradient. Change "The gradient is" to "The normalized gradient is". I think that is all that is needed, because the rest can be solved without actually computing anything. I would just do this.
  3. Change the problem and solution to use the unnormalized gradient. This only affects the first half of the solution.

@mykelk
Copy link
Contributor

mykelk commented May 1, 2022

Let's go with 3.

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

3 participants