Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove CLV deprecation warnings #1516

Open
arthurmello opened this issue Feb 22, 2025 · 1 comment
Open

Remove CLV deprecation warnings #1516

arthurmello opened this issue Feb 22, 2025 · 1 comment
Labels
CLV deprecation good first issue Good for newcomers . Doesn't require extensive knowledge of the repo and package

Comments

@arthurmello
Copy link
Contributor

Remove:

  • _rename_posterior_variables and deprecated_keys = [key for key in model_config if key.endswith("_prior")], in basic CLV model
  • the related tests (such as test_backward_compatibility_with_old_config and test_deprecation_warning_on_old_config)
@arthurmello arthurmello mentioned this issue Feb 22, 2025
Merged
6 tasks
@ColtAllen ColtAllen added the good first issue Good for newcomers . Doesn't require extensive knowledge of the repo and package label Mar 3, 2025
@ColtAllen ColtAllen changed the title Remove deprecation warnings and backwards compatibility after suppression of _prior suffix from CLV models Remove CLV deprecation warnings Mar 3, 2025
@ColtAllen
Copy link
Collaborator

Adding additional deprecations to this; will update as PRs are merged.

  • clv.utils.clv_summary

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
CLV deprecation good first issue Good for newcomers . Doesn't require extensive knowledge of the repo and package
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ColtAllen @arthurmello @wd60622