Make nsga2 tests optional + docs update

[ewu63]

Purpose

Closes #345. Also updated docs slightly + added an nsga2 test to the Rosenbrock problem.

Expected time until merged

1 week

Type of change

• Bugfix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (non-backwards-compatible fix or feature)
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no API changes)
• Documentation update
• Maintenance update
• Other (please describe)

Testing

Checklist

• I have run flake8 and black to make sure the Python code adheres to PEP-8 and is consistently formatted
• I have formatted the Fortran code with fprettify or C/C++ code with clang-format as applicable
• I have run unit and regression tests which pass locally with my changes
• I have added new tests that prove my fix is effective or that my feature works
• I have added necessary documentation

[codecov]

Codecov Report

Merging #351 (7391afb) into main (7e01f2c) will decrease coverage by 11.92%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #351       +/-   ##
===========================================
- Coverage   84.42%   72.51%   -11.92%     
===========================================
  Files          22       22               
  Lines        3307     3307               
===========================================
- Hits         2792     2398      -394     
- Misses        515      909      +394     

see 7 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[ewu63]

Bump @lamkina

[marcomangano]

I am sorry to have checked this too late @nwu63 , but I don't really get the point of this step here. Can't we remove this altogether? We are already skipping all n_obj == 2 tests by default on all platforms. Is this related to an additional test that testflo runs on windows?
Locally I only have the last two tests of this stack:

about to run test_nsga2_multi_objective.py:TestNSGA2.test_opt .\test_nsga2_multi_objective.py:TestNSGA2.test_opt  ... SKIP (00:00:0.00, 0 MB)
test_nsga2_multi_objective.py fails on windows with two objectives! Skipping for now.
    about to run test_nsga2_multi_objective.py:TestNSGA2.test_opt_0 .\test_nsga2_multi_objective.py:TestNSGA2.test_opt_0 ... OK (00:00:4.17, 0 MB)
    about to run test_nsga2_multi_objective.py:TestNSGA2.test_opt_1 .\test_nsga2_multi_objective.py:TestNSGA2.test_opt_1  ... SKIP (00:00:0.00, 0 MB)
skip flaky NSGA2 tests

[ewu63]

This skips all tests on Windows, the below skips all multiobjective tests, so the only tests being run are the single-objective tests on linux.

[marcomangano]

I understand the idea (and that is what I was expecting) but that printout in my comment is from the Windows test log - apparently the single-objective test is running on Windows? Or is it a Testflo fluke?

[ewu63]

It is possible that the skipIf doesn't play nicely with parameterized. However this is actually what we want---the single objective test should pass on Windows and we should test it, while the multi-objective test appears to be flat out broken. Let me throw together a PR to update this.