Make TimingModel.params and TimingModel.ordered_params identical.

[abhisrkckl]

See #1295

These parameter lists had different orders because Python dicts were unordered until Python 3.7. Since this is no longer the case, TimingModel.params and TimingModel.ordered_params have no reason to be different anymore.

[codecov]

Codecov Report

Patch coverage: 78.57% and project coverage change: -0.01% ⚠️

Comparison is base (dc40e8b) 68.14% compared to head (18f4604) 68.14%.
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1617      +/-   ##
==========================================
- Coverage   68.14%   68.14%   -0.01%     
==========================================
  Files          98       98              
  Lines       22660    22658       -2     
  Branches     3893     3892       -1     
==========================================
- Hits        15442    15440       -2     
  Misses       6264     6264              
  Partials      954      954              

Files Changed	Coverage Δ
src/pint/models/timing_model.py	83.57% <75.00%> (-0.03%)	⬇️
src/pint/fitter.py	84.05% <100.00%> (ø)

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

[dlakaplan]

Would it be better to just remove the ordered_params and make all instances params?

[abhisrkckl]

I think we should keep it for the time being since other people's codes may depend on it. We can deprecate it and then remove it in the future once people get enough time to change their code.

[dlakaplan]

OK, I think I can merge this when you are done.

[abhisrkckl]

I am done.