Remove `syms` and `paramsyms` arguments for `OptimizationFuntion`

sunxd3 commented 9 months ago

Checklist

[ ] Appropriate tests were added
[ ] Any code changes were done in a way that does not break public API
[ ] All documentation related to code changes were updated
[ ] The new code follows the contributor guidelines, in particular the SciML Style Guide and COLPRAC.
[ ] Any new documentation only uses public API

Additional context

In this PR, syms and paramsyms fields are removed from OptimizationFunction.

codecov[bot] commented 9 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (8d53748) 0.11% compared to head (b160d13) 7.74%. Report is 2 commits behind head on master.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #644 +/- ## ========================================= + Coverage 0.11% 7.74% +7.63% ========================================= Files 40 40 Lines 2689 2699 +10 ========================================= + Hits 3 209 +206 + Misses 2686 2490 -196 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

ChrisRackauckas commented 9 months ago

Handled in https://github.com/SciML/Optimization.jl/pull/626

SciML / Optimization.jl