Skip to content

deprecate: addProfileGenerator deprecation#146

Merged
sbillinge merged 6 commits intodiffpy:v3.3.0from
cadenmyers13:addprofilegenerator-dep
Feb 7, 2026
Merged

deprecate: addProfileGenerator deprecation#146
sbillinge merged 6 commits intodiffpy:v3.3.0from
cadenmyers13:addprofilegenerator-dep

Conversation

@cadenmyers13
Copy link
Contributor

@cadenmyers13 cadenmyers13 commented Feb 6, 2026

deprecate FitContribution.addProfileGenerator. Dont merge until upstream v4.0.0 branch has been made.

@cadenmyers13
Copy link
Contributor Author

@sbillinge ready for review. Before merge though, do we want to create a new branch v4.0.0? then we can do all deprecation into there?

@codecov
Copy link

codecov bot commented Feb 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.44%. Comparing base (8db7cc6) to head (92e8158).
⚠️ Report is 15 commits behind head on v3.3.0.

Additional details and impacted files
@@            Coverage Diff             @@
##           v3.3.0     #146      +/-   ##
==========================================
+ Coverage   69.36%   69.44%   +0.07%     
==========================================
  Files          25       25              
  Lines        3457     3466       +9     
==========================================
+ Hits         2398     2407       +9     
  Misses       1059     1059              
Files with missing lines Coverage Δ
tests/test_contribution.py 99.53% <100.00%> (+0.02%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sbillinge sbillinge changed the base branch from main to v3.3.0 February 6, 2026 19:54
@sbillinge sbillinge merged commit d3c210e into diffpy:v3.3.0 Feb 7, 2026
7 checks passed
@sbillinge
Copy link
Contributor

@cadenmyers13 I am merging into v3.3.0 as the base branch. Please could you use this as base? 4.0.0 will be the version where we remove the old functions, we want to deprectate for a few minor release bumps and a year or so.

@cadenmyers13
Copy link
Contributor Author

@sbillinge yep! Sorry I meant the minor bump not major bump you're right

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