Fix empty hist

[ewu63]

Purpose

If the initial evaluation fails for SNOPT, exit 61 is returned. However, this results in an empty history file with major=True, which errors out:

  File ".../pyoptsparse/pyOpt_history.py", line 654, in getValues
    data[name] = np.stack(data[name], axis=0)
  File ".../numpy/core/shape_base.py", line 445, in stack
    raise ValueError('need at least one array to stack')

This PR fixes this issue.

Expected time until merged

2 days

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

[ewu63]

Also would appreciate a patch version bump (preferably in this PR)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.15%. Comparing base (0fd3dd7) to head (a09162c).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #399      +/-   ##
==========================================
+ Coverage   74.10%   74.15%   +0.05%     
==========================================
  Files          22       22              
  Lines        3294     3297       +3     
==========================================
+ Hits         2441     2445       +4     
+ Misses        853      852       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[marcomangano]

LGTM. Can you confirm you added __all__ for linting purposes?

[ewu63]

LGTM. Can you confirm you added __all__ for linting purposes?

Yep, I believe that's a PEP8 requirement (to avoid "unused imports").