llnl / llnl/SNLS

Add new tests for our N-dimension solvers

Open
#5 0 comments 0 reactions 1 assignee View on GitHub

@rcarson3 is already working on this.

Since Sep 2, 2021.

enhancement Testing
Dominant language
C++
Stars
18
Forks
3
PR merge metrics
No merged PRs in 30d

Description

For our N-dimension solvers, we should look at maybe testing our solvers out with some of the test suites given here: https://github.com/devernay/cminpack/tree/master/examples and here: https://people.sc.fsu.edu/~jburkardt/f_src/test_nonlin/test_nonlin.html . These tests are from the MINPACK test suite for the nonlinear solvers and should at least provide confidence in any new N-dimension solvers that we might add. I only mention this as I've found that the Broyden test didn't catch an error in a newer solver I'm working on but one of the above tests did.

I can look at adding at least some of those with my up-coming PR for this new solver.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.