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

Create confidence_intervals module #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jcostello-atlassian
Copy link

This creates a confidence_intervals module with a function that calculates Wilson score binomial proportion confidence intervals.

This creates a confidence_intervals module with a function
that calculates Wilson score binomial proportion confidence
intervals.
@jcostello-atlassian
Copy link
Author

Hi @tphan-splk, I noticed you're the last person to commit to this repo. Would you be able to review this PR? Thanks!

@joe-sfx
Copy link
Contributor

joe-sfx commented Jul 17, 2023

Hi @jcostello-atlassian and thank you for the suggestion! We will need to spend some time on the organization for this type of functionality and details on the signature. We loosely try to mimic NumPy where possible.

We also need a general policy/procedure re: external contributions as this GitHub repository merely reflects what is deployed in Splunk Infrastructure Monitoring (fka SignalFx) realms and is intended as documentation for both the "SignalFlow standard library" as well as the SignalFlow language itself; there is no pipeline/mechanism going from this repository to any deployment.

@jcostello-atlassian
Copy link
Author

Sure, happy to refactor to align things more. I was lightly using https://www.statsmodels.org/dev/generated/statsmodels.stats.proportion.proportion_confint.html as inspiration, but didn't realize the NumPy precedent. Is there a specific function signature you can share that we can emulate here?

As for the deployment pipeline and external contributions, is that something realistically in scope or feasible at this time? I wasn't aware the repo wasn't set up for external contributions, and just wanted to share some functionality for others that's been helpful for us. Also importing the function from the stats library would allow us to DRY things up a bit on our end 😄.

So yeah, happy to touch things up if there's a path to deployment, but otherwise am also fine with leaving this as a feature request for now.

@jcostello-atlassian
Copy link
Author

Hey @joe-sfx, just wanted to follow up here. Would it be preferable if I created a feature request for SignalFx/Splunk?

@joe-sfx
Copy link
Contributor

joe-sfx commented Sep 15, 2023

Hi, sorry for the delay. Yes, please create a feature request. Thank you!

@jcostello-atlassian
Copy link
Author

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