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.

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

mark binomial_exact() function for removal #1731

Open
GavinHuttley opened this issue Feb 24, 2024 · 1 comment
Open

mark binomial_exact() function for removal #1731

GavinHuttley opened this issue Feb 24, 2024 · 1 comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@GavinHuttley
Copy link
Collaborator

GavinHuttley commented Feb 24, 2024

What should be changed

cogent3.maths.stats.binomial_exact()

Why

Not used in cogent3, available in scipy.stats?

How much notice

6 months notice. This should be represented in the user warning as a specific version. e.g. .<current month + x> where x is from the PR date.

Guidelines for deprecation

As this is a function that will be removed, see dev guidelines for deprecation of code. Indicate the limiting version (for when the old function will be removed) as the number above from when you make the change.

@GavinHuttley GavinHuttley added help wanted Extra attention is needed good first issue Good for newcomers labels Feb 24, 2024
@GavinHuttley GavinHuttley changed the title mark binomial_exact function for removal mark binomial_exact() function for removal Feb 24, 2024
@ChangsenJiang
Copy link

I would like to do this issue.

ChangsenJiang added a commit to ChangsenJiang/cogent3 that referenced this issue Mar 13, 2024
cogent3#1731

In this commit, the binomial_exact() function was removed and the related test_binomial_series(),  test_binomial_exact(), test_binomial_exact_floats(), and test_binomial_exact_errors() were also been removed
ChangsenJiang added a commit to ChangsenJiang/cogent3 that referenced this issue Mar 13, 2024
cogent3#1731

In this new commit, we revert the change within the function to the original, and delete the new import
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants