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 poisson_low() function for removal #1733

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

mark poisson_low() function for removal #1733

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.poisson_low()

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
@Zongjing-Han
Copy link
Contributor

I would like to do this issue

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

No branches or pull requests

2 participants