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

Fix "std dev" link #204

Merged
merged 2 commits into from
Sep 23, 2024
Merged

Fix "std dev" link #204

merged 2 commits into from
Sep 23, 2024

Conversation

vstinner
Copy link
Member

No description provided.

@fzakaria
Copy link

Does it help just to emit the standard deviation?

I was reading that knowing the standard deviation for a sample set isn't very helpful; I think it gives off the wrong impression that it's used as a confidence interval.

Reading the link - my feeling was the intent was for it to be a 95% confidence interval especially since the t-test value was calcualte.

@vstinner
Copy link
Member Author

The PR fix the documentation of the current implementation: describe correctly what's being used.

@fzakaria
Copy link

fzakaria commented Sep 16, 2024

sorry i wasn't clear; the output of stats has a +- using the standard deviation; that gives the impression like it's the confidence interval no?

A title was too short
@vstinner
Copy link
Member Author

cc @corona10

@corona10
Copy link
Member

I will take a look during the sprint.

Copy link
Member

@corona10 corona10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@corona10 corona10 merged commit 922ad35 into main Sep 23, 2024
13 checks passed
@vstinner vstinner deleted the stdev_doc branch September 23, 2024 18:54
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.

3 participants