Skip to content

Comments

Deprecate: Deprecate setResidualEquation and getResidualEquation in Fitcontribution#156

Merged
sbillinge merged 5 commits intodiffpy:v3.3.0from
cadenmyers13:fitcontrib-dep-final
Feb 20, 2026
Merged

Deprecate: Deprecate setResidualEquation and getResidualEquation in Fitcontribution#156
sbillinge merged 5 commits intodiffpy:v3.3.0from
cadenmyers13:fitcontrib-dep-final

Conversation

@cadenmyers13
Copy link
Contributor

These are the final deprecation in FitContribution. Also I missed tagging a deprecated function in SimpleRecipe so I added it here. This slipped by because there are no direct tests for SimpleRecipe

@cadenmyers13
Copy link
Contributor Author

@sbillinge ready for review

@codecov
Copy link

codecov bot commented Feb 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.63%. Comparing base (7443c6a) to head (1a3aa78).
⚠️ Report is 7 commits behind head on v3.3.0.

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              
Files with missing lines Coverage Δ
tests/test_contribution.py 99.58% <100.00%> (+0.01%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sbillinge
Copy link
Contributor

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 sbillinge merged commit 31e9698 into diffpy:v3.3.0 Feb 20, 2026
6 checks passed
@cadenmyers13 cadenmyers13 deleted the fitcontrib-dep-final branch February 20, 2026 15:21
@cadenmyers13
Copy link
Contributor Author

@sbillinge okay got it. Will do moving forward. Thanks!

@sbillinge
Copy link
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants