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 12
    • Merge requests 12
  • 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
  • ift
  • NIFTy
  • Merge requests
  • !550

Try to avoid crashes in the line searcher

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Martin Reinecke requested to merge paranoid_line_search into NIFTy_6 Jun 24, 2020
  • Overview 17
  • Commits 3
  • Pipelines 3
  • Changes 1

With this patch, the line searcher checks for floating point errors, infinities and NaNs when extending its search interval. If it encounters one of those, it reduces the step size and tries again repeatedly. This actually seems to avoid crashes in real-world applications.

@all: If you have problems with minimizations aborting due to overflow etc., please give this one a try and report whether or not it works for you!

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