Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • N NIFTy
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 19
    • Issues 19
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 14
    • Merge requests 14
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar

On Thursday, 2nd February from 9 to 10.00 am there will be a maintenance with a short downtime of the GitLab service.

  • ift
  • NIFTy
  • Merge requests
  • !188

normalize descent direction for SteepestDescent and VL_BFGS (but not for RelaxedNewton!)

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Martin Reinecke requested to merge normalize_descent into master Aug 17, 2017
  • Overview 4
  • Commits 5
  • Pipelines 5
  • Changes 1

Following a discussion with Jakob, it appears that the length of the descent direction vector only has real meaning for RelaxedNewton. The other two minimizers can get into trouble (e.g. by going to extreme coordinates and causing overflows) when suplied with very long descent vectors, so I'm normalizing them. A good step length will hopefully be found by the line search algorithm.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: normalize_descent