Deprecate: Deprecate setResidualEquation and getResidualEquation in Fitcontribution#156
Conversation
|
@sbillinge ready for review |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## v3.3.0 #156 +/- ##
==========================================
+ Coverage 70.55% 70.63% +0.07%
==========================================
Files 25 25
Lines 3597 3606 +9
==========================================
+ Hits 2538 2547 +9
Misses 1059 1059
🚀 New features to boost your workflow:
|
|
Thanks for this @cadenmyers13 I will merge. As a general matter, whenever we touch code we want to improve it, and since we are moving in the direction of using pytest for everything, when you do write new tests it is a good idea to write them using pytest so we don't add to legacy unittest tests that we may have to fix in the future..... |
|
@sbillinge okay got it. Will do moving forward. Thanks! |
|
This general philosophy applies to any update. For example, we now use pathlib for all our path munging. We don't want to go back through all our code to make that fix since everything is working, but if you touch a function for some other reason (deprecating it), it is good practice to make any obvious updates just on that function. Pathlib, but also anything else you notice that brings it up to current group standards. Of course, if it starts to break it, let's think again. |
These are the final deprecation in
FitContribution. Also I missed tagging a deprecated function inSimpleRecipeso I added it here. This slipped by because there are no direct tests forSimpleRecipe