Bugfixes in ParmEst and DoE

[djlaky]

Fixes #3372 .

Fixed error in #3372 with suggestion. Ordering appears to be deterministic with new convention.

Also fixed two bugs within pyomo.doe.

Summary/Motivation:

Improving code quality and consistency through bugfixes.

Changes proposed in this PR:

• Implemented change suggested in Nondeterminism bug in parmest #3372
• Reverted parameter values in _sequentia_fim function in pyomo.doe to correctly calculate the FIM with finite differencing
• Changed get_<> functions to be correct. Some were reporting incorrect error messages and populating incorrect values for the results object.
• Updated testing function to reflect changes. (Testing suffered from the same copy-paste error)

Legal Acknowledgement

By contributing to this software project, I have read the contribution guide and agree to the following terms and conditions for my contribution:

1. I agree my contributions are submitted under the BSD license.
2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

[mrmundt]

FYI, @djlaky , we are seeing lots of intermittent failures on Windows/conda setup. Not related to your changes.

[djlaky]

Thanks @mrmundt. This should be ready for review, @blnicho, @jsiirola, @mrmundt.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.65%. Comparing base (5d96b24) to head (679f269).
Report is 29 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3512   +/-   ##
=======================================
  Coverage   88.64%   88.65%           
=======================================
  Files         881      881           
  Lines      100918   100919    +1     
=======================================
+ Hits        89461    89467    +6     
+ Misses      11457    11452    -5     

Flag	Coverage Δ
builders	26.64% <0.00%> (+<0.01%)	⬆️
default	84.72% <100.00%> (?)
expensive	34.11% <33.33%> (?)
linux	86.08% <100.00%> (-2.31%)	⬇️
linux_other	86.08% <100.00%> (+<0.01%)	⬆️
osx	76.11% <33.33%> (-0.01%)	⬇️
win	84.56% <100.00%> (+<0.01%)	⬆️
win_other	84.56% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[jsiirola]

This is fine, but in the future, DoE needs to be gone through and wrapped to 88 columns (for style consistency / black)

[djlaky]

This is fine, but in the future, DoE needs to be gone through and wrapped to 88 columns (for style consistency / black)

Sure thing. I will add this to the list of To-Dos (#3345)