fixed convertJacobian

[gjkennedy]

call when jacType == "csr" so that it retuns CSR data instead of passing through

Purpose

Minor bug fix

Expected time until merged

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

I checked it using ParOptSparse which requires the Jacobian matrix in a CSR structure

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

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (5fef213) 73.34% compared to head (b97db72) 61.60%.

Files	Patch %	Lines
pyoptsparse/pyOpt_optimizer.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main     #371       +/-   ##
===========================================
- Coverage   73.34%   61.60%   -11.74%     
===========================================
  Files          22       22               
  Lines        3316     3318        +2     
===========================================
- Hits         2432     2044      -388     
- Misses        884     1274      +390     

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

[marcomangano]

Sorry for the delay, LGTM